aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | | | | | * | | | | | | | Remove sir lancebot names from seasonsGravatar ks1292021-01-06-7/+0
| | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | Remove unnecessary pass statementGravatar ks1292021-01-06-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Dennis Pham <[email protected]>
| | | | | | | * | | | | | | | Re-lock PipfileGravatar Karlis S2021-01-05-4/+4
| | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | Merge branch 'master' into branding-managerGravatar ks1292021-01-05-13/+17
| | | | | | | |\ \ \ \ \ \ \ \ | | |_|_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | |
| | | | | | | * | | | | | | | Remove unnecessary Keys import from utils cogGravatar ks1292021-01-03-1/+1
| | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | Merge branch 'master' into branding-managerGravatar Karlis S2021-01-03-483/+549
| | | | | | | |\ \ \ \ \ \ \ \ | | | | | | | |/ / / / / / / / | | | | | | |/| | | | | | | |
| | | | | | | * | | | | | | | Re-lock PipfileGravatar Karlis S2020-12-17-142/+142
| | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | Merge branch 'master' into branding-managerGravatar ks1292020-12-17-1432/+1943
| | | | | | | |\ \ \ \ \ \ \ \
| | | | | | | * | | | | | | | | Fix wrong import ordersGravatar ks1292020-11-17-2/+2
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Use GitHub token for fetching PEPsGravatar ks1292020-11-17-2/+7
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Move GitHub API key to better locationGravatar ks1292020-11-17-13/+5
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Port branding management to this bot from SeasonalBotGravatar ks1292020-11-17-0/+569
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Handle branding error in error handlerGravatar ks1292020-11-17-2/+6
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Create mock_in_debug decorator for brandingGravatar ks1292020-11-17-1/+22
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Add required colors and emoji for branding managementGravatar ks1292020-11-17-0/+12
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Create AssetType enumGravatar ks1292020-11-16-0/+11
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Add arrow dependency for brandingGravatar ks1292020-11-16-1/+10
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Create config for tokens and add GitHub tokenGravatar ks1292020-11-16-0/+10
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Add basic branding configurationGravatar ks1292020-11-16-0/+10
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Port seasons from SeasonalBot to hereGravatar ks1292020-11-16-0/+181
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Add BrandingError to errorsGravatar ks1292020-11-16-0/+6
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | Add months Enum to constantsGravatar ks1292020-11-16-1/+19
| | | | | | | | | | | | | | | |
| | | | | | | | | * | | | | | | Move not allowed to run message to constant and match comment with codeGravatar ks1292020-12-02-4/+6
| | | | | | | | | | | | | | | |
| | | | | | | | | * | | | | | | Don't log CommandError, handle it as check failGravatar ks1292020-11-21-2/+2
| | | | | | | | | | | | | | | |
| | | | | | | | | * | | | | | | Fix import order of help command fileGravatar ks1292020-11-07-1/+1
| | | | | | | | | | | | | | | |
| | | | | | | | | * | | | | | | Catch CommandError for help command can_run awaitGravatar ks1292020-11-07-5/+9
| | | | | | | | | | | | | | | |
| | | | | | | | | * | | | | | | Merge branch 'master' into help-disabled-commandGravatar ks1292020-11-07-1496/+2971
| | | | | | | | | |\ \ \ \ \ \ \
| | | | | | | | | * | | | | | | | Help: Show different message for case when command is disabledGravatar ks1292020-09-28-2/+5
| | | | | | | | | | | | | | | | |
| | | | | | | | | * | | | | | | | Help: Add handling of disabled commands to avoid DisabledCommand errorGravatar ks1292020-09-27-1/+1
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | Set asyncio logger level to INFOGravatar MarkKoz2021-01-04-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If asyncio's debug mode is enabled, the asyncio logger's level gets set to DEBUG. While other features of the debug mode are useful, the DEBUG log level spams generally irrelevant stuff.
| | | | | | | | | | | | | * | | | HelpChannels: fix unclaim exiting too early if claimant is NoneGravatar MarkKoz2021-01-04-4/+2
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | HelpChannels: fix manual use of lock decoratorGravatar MarkKoz2021-01-04-3/+3
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | HelpChannels: fix race condition when unclaiming a channelGravatar MarkKoz2021-01-04-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Place a channel-specific lock on `unclaim_channel`. If both the dormant task and the command simultaneously unclaim a channel, one of them will silently be aborted. Fix #1341
| | | | | | | | | | | | | * | | | HelpChannels: change caller str parameter to a boolGravatar MarkKoz2021-01-04-11/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Booleans are less error-prone than strings.
| | | | | | | | | | | | | * | | | HelpChannels: unschedule the dormant task in unclaim_channelGravatar MarkKoz2021-01-04-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ensure the cancellation will be under the lock once the lock is added.
| | | | | | | | | | | | | * | | | HelpChannels: clarify close command docstringGravatar MarkKoz2021-01-04-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Referencing internal functions in public-facing documentation is not helpful to users.
| | | | | | | | | | | | | * | | | HelpChannels: refactor the close command checkGravatar MarkKoz2021-01-04-10/+10
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | HelpChannels: make a channel available within the lockGravatar MarkKoz2021-01-04-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the lock causes the function to abort, a new channel shouldn't be made available. However, the only way to know it's aborted from the outside would be through a return value or global variable. Neither seem as nice as just just using `create_task` within the lock to avoid having `move_to_available` hold the lock.
| | | | | | | | | | | | | * | | | Add asyncio.create_task wrapper which logs exceptionsGravatar MarkKoz2021-01-04-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Normally exceptions are only logged when tasks are garbage collected. This wrapper will allow them to be logged immediately through a done callback. This is similar to how the Scheduler logs errors.
| | | | | | | | | | | | | * | | | HelpChannels: fix race condition between claiming and unclaimingGravatar MarkKoz2021-01-04-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The race condition is when a user claims a channel while their other channel is being unclaimed. Specifically, it's while their cooldown is being removed. The lock ensures that either the cooldown will be re-applied after it's removed or that it won't be removed since `unclaim_channel` will see the user has another claimed channel.
| | | | | | | | | | | | | * | | | Lock: support waiting until a lock is availableGravatar MarkKoz2021-01-04-37/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's been determined that asyncio.Lock is safe to use in such manner. Therefore, replace LockGuard entirely with asyncio.Lock.
| | | | | | | | | | | | | * | | | HelpChannels: refactor channel unclaimingGravatar MarkKoz2021-01-04-20/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Narrow the scope of `move_to_dormant` to just moving the channel. Following the design of `claim_channel`, make `unclaim_channel` handle cooldowns and unpinning.
| | | | | | | | | | | | | * | | | HelpChannels: remove obsolete channel position trace logGravatar MarkKoz2021-01-04-1/+0
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | HelpChannels: refactor stat trackingGravatar MarkKoz2021-01-04-27/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move significant code related to stats to a separate module.
| | | | | | | | | | | | | * | | | HelpChannels: move function to the channel moduleGravatar MarkKoz2021-01-04-42/+45
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | HelpChannels: prevent user from claiming multiple channels quicklyGravatar MarkKoz2021-01-04-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's conceivable for a user to be able to quickly send a message in all available channels before the code has a chance to add the cooldown role. Place a lock on the author to prevent the claim code from running multiple times for the same user.
| | | | | | | | | | | | | * | | | HelpChannels: await init task at the start of event listenersGravatar MarkKoz2021-01-04-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It feels safer to do this since the init task moves channels to different categories and the listeners check if channels are in certain categories.
| | | | | | | | | | | | | * | | | HelpChannels: use a more accurate lock for messagesGravatar MarkKoz2021-01-04-47/+35
| | |_|_|_|_|_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the `lock_arg` decorator to keep a separate lock for each channel rather than a single lock used by all messages. Separate the core logic in `on_message` into a separate function to facilitate the use of `lock_arg` - not everything in `on_message` needs to be under the lock.
| * | | | | | | | | | | | | | | Merge pull request #1340 from python-discord/ks123/raw-fixGravatar Matteo Bertucci2021-01-04-2/+6
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add check does user have permission to get raw message and re-enable raw command
| | * \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into ks123/raw-fixGravatar Matteo Bertucci2021-01-04-71/+131
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-28 09:45:18 +0000