Skip to content

Remove GlobalCooldown dead code#36

Open
Copilot wants to merge 3 commits intomainfrom
copilot/fix-user-cooldown-issue
Open

Remove GlobalCooldown dead code#36
Copilot wants to merge 3 commits intomainfrom
copilot/fix-user-cooldown-issue

Conversation

Copy link
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
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