Skip to content

Remove GlobalCooldown dead code#36

Merged
FlorentPoinsaut merged 3 commits intomainfrom
copilot/fix-user-cooldown-issue
Mar 24, 2026
Merged

Remove GlobalCooldown dead code#36
FlorentPoinsaut merged 3 commits intomainfrom
copilot/fix-user-cooldown-issue

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 15, 2026

GlobalCooldown became dead code after the guess command was switched to per-user CooldownManager. Nothing in production references it anymore.

Changes

  • bot/cooldown.py — deleted GlobalCooldown class
  • tests/test_cooldown.py — removed TestGlobalCooldownInit, TestGlobalCooldownIsOnCooldown, TestGlobalCooldownSetDuration, and the now-unused GlobalCooldown import

bot/cooldown.py reaches 100% test coverage after removal.

Original prompt

This section details on the original issue you should resolve

<issue_title>Cooldown seems not be by user but for all</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: FlorentPoinsaut <1256948+FlorentPoinsaut@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix cooldown functionality to be user-specific Fix guess cooldown to be per-user instead of global Mar 15, 2026
Copilot AI requested a review from FlorentPoinsaut March 15, 2026 09:52
@FlorentPoinsaut FlorentPoinsaut marked this pull request as ready for review March 17, 2026 09:55
Co-authored-by: FlorentPoinsaut <1256948+FlorentPoinsaut@users.noreply.github.com>
Copilot AI changed the title Fix guess cooldown to be per-user instead of global Remove GlobalCooldown dead code Mar 17, 2026
@FlorentPoinsaut FlorentPoinsaut merged commit b575627 into main Mar 24, 2026
2 checks passed
@FlorentPoinsaut FlorentPoinsaut deleted the copilot/fix-user-cooldown-issue branch March 24, 2026 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cooldown seems not be by user but for all

2 participants