| Commit message (Collapse) | Author | Age | Lines | ||
|---|---|---|---|---|---|
| ... | |||||
| | * | | | Add Silence cog load to docstring. | 2020-03-15 | -1/+1 | ||
| | | | | | |||||
| | * | | | Move coro execution outside of if condition. | 2020-03-15 | -8/+6 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This gives us a clearer look at the general flow control and what's getting executed. Comment was also moved to its relevant line. Co-authored-by: MarkKoz <[email protected]> | ||||
| | * | | | Adjust docstring styling. | 2020-03-15 | -2/+2 | ||
| | | | | | | | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]> | ||||
| | * | | | Use dict instead of a set and custom class. | 2020-03-15 | -44/+8 | ||
| | | | | | | | | | | | | | | | | | | | | | The FirstHash class is no longer necessary with only channels and the current loop in tuples. FirstHash was removed, along with its tests and tests were adjusted for new dict behaviour. | ||||
| | * | | | Add tests for `_notifier`. | 2020-03-11 | -0/+19 | ||
| | | | | | |||||
| | * | | | Improve commands help. | 2020-03-11 | -3/+5 | ||
| | | | | | |||||
| | * | | | Add logging to commands. | 2020-03-11 | -0/+3 | ||
| | | | | | |||||
| | * | | | Change various logging levels. | 2020-03-11 | -7/+7 | ||
| | | | | | |||||
| | * | | | Add tests for `remove_channel`. | 2020-03-11 | -0/+18 | ||
| | | | | | |||||
| | * | | | Remove `_current_loop` from setup. | 2020-03-11 | -2/+1 | ||
| | | | | | |||||
| | * | | | Add tests for `add_channel`. | 2020-03-11 | -0/+21 | ||
| | | | | | |||||
| | * | | | Create test case for `SilenceNotifier` | 2020-03-11 | -1/+7 | ||
| | | | | | |||||
| | * | | | Add tests for `_get_instance_vars`. | 2020-03-11 | -1/+28 | ||
| | | | | | |||||
| | * | | | Pass created channel instead of new object. | 2020-03-11 | -1/+1 | ||
| | | | | | | | | | | | | | | | | | Creating a new object caused the assert to fail because different objects were used. | ||||
| | * | | | Reset `self.ctx` call history after every subtest. | 2020-03-11 | -0/+1 | ||
| | | | | | |||||
| | * | | | Fix erroneous `assert_called_once_with` calls. | 2020-03-11 | -8/+7 | ||
| | | | | | | | | | | | | | | | | | `assert_called_once_with` was being tested on call_args which always reported success.st. | ||||
| | * | | | Add test for `cog_check`. | 2020-03-11 | -0/+7 | ||
| | | | | | |||||
| | * | | | Add test for `cog_unload`. | 2020-03-11 | -1/+17 | ||
| | | | | | |||||
| | * | | | Use patch decorators and assign names from `with` patches. | 2020-03-11 | -11/+11 | ||
| | | | | | |||||
| | * | | | Separate tests for permissions and `muted_channels.add` on `_silence`. | 2020-03-11 | -5/+8 | ||
| | | | | | |||||
| | * | | | Add tests for `_unsilence` | 2020-03-11 | -1/+33 | ||
| | | | | | |||||
| | * | | | Use async test case. | 2020-03-11 | -14/+12 | ||
| | | | | | | | | | | | | | | | | | This allows us to use coroutines with await directly instead of asyncio.run | ||||
| | * | | | Add test for `_silence` method. | 2020-03-11 | -1/+34 | ||
| | | | | | |||||
| | * | | | Move adding of channel to `muted_channels` up. | 2020-03-11 | -1/+1 | ||
| | | | | | | | | | | | | | | | | | Before the channel was not added if `persistent` was `True`. | ||||
| | * | | | Remove channel mentions from output discord messages. | 2020-03-11 | -8/+8 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | With the removal of the channel args, it's no longer necessary to mention the channel in the command output. Tests adjusted accordingly | ||||
| | * | | | Remove unnecessary args from test cases. | 2020-03-11 | -6/+6 | ||
| | | | | | | | | | | | | | | | | | | | | | Needless call args which were constant were kept in the test cases, resulting in redundant code, the args were moved directly into the function call. | ||||
| | * | | | Add alert with silenced channels on `cog_unload`. | 2020-03-10 | -0/+11 | ||
| | | | | | |||||
| | * | | | Remove `channel` arg from commands. | 2020-03-10 | -20/+10 | ||
| | | | | | |||||
| | * | | | Retain previous channel overwrites. | 2020-03-10 | -4/+12 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously silencing a channel reset all overwrites excluding `send_messages` and unsilencing them removed all overwrites. This is prevented by getting the current overwrite and applying it with only send_messages changed. | ||||
| | * | | | Use `patch.object` instead of patch with direct `return_value`. | 2020-03-09 | -9/+2 | ||
| | | | | | |||||
| | * | | | Move notifier to separate class. | 2020-03-08 | -40/+46 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Separating the notifier allows us to keep the Silence class and its methods to be more focused on the class' purpose, handling the logic of adding/removing channels and the loop itself behind `SilenceNotifier`'s interface. | ||||
| | * | | | Remove "Channel" from output string for consistency. | 2020-03-08 | -1/+1 | ||
| | | | | | |||||
| | * | | | Use `Context.invoke` instead of calling `unsilence` directly. | 2020-03-08 | -1/+1 | ||
| | | | | | | | | | | | | | | | | | | | | | Calling the command coro directly did unnecessary checks and made tests for the method harder to realize. | ||||
| | * | | | Add test for `unsilence` discord output. | 2020-03-08 | -0/+9 | ||
| | | | | | |||||
| | * | | | Add test for `silence` discord output. | 2020-03-08 | -0/+24 | ||
| | | | | | |||||
| | * | | | Add test case for Silence cog. | 2020-03-08 | -1/+10 | ||
| | | | | | |||||
| | * | | | Add tests for FirstHash class. | 2020-03-08 | -0/+25 | ||
| | | | | | |||||
| | * | | | Add handling for shh/unshh for `CommandNotFound`. | 2020-03-08 | -4/+32 | ||
| | | | | | |||||
| | * | | | Restrict cog to moderators. | 2020-03-08 | -1/+7 | ||
| | | | | | |||||
| | * | | | Add logging to loop start and loop end. | 2020-03-02 | -0/+8 | ||
| | | | | | |||||
| | * | | | Add Silence cog. | 2020-03-02 | -0/+143 | ||
| | | | | | | | | | | | | | | | | | FirstHash is used for handling channels in `loop_alert_channels` set as tuples without considering other elements. | ||||
| | * | | | Add HushDurationConverter. | 2020-03-02 | -0/+28 | ||
| | | | | | |||||
| | | * | | Increase syncer logging level | 2020-03-28 | -1/+1 | ||
| | | | | | |||||
| | | * | | Use debug log level instead of warning in `post_user` | 2020-03-28 | -1/+1 | ||
| | | | | | |||||
| | | * | | Logging severity pass from review | 2020-03-28 | -4/+4 | ||
| | | | | | |||||
| | | * | | Initial pass on log severity reduction | 2020-03-26 | -9/+8 | ||
| | | | | | | | | | | | | | | | | | | | | | With the updated definition on logging levels, there are a few events that were issuing logs at too high of a level. This also includes some kaizening of existing log messages. | ||||
| | | | * | Merge branch 'master' into zen-match-fix | 2020-03-26 | -284/+320 | ||
| | | | |\ | |_|_|/ |/| | | | |||||
| * | | | | Merge pull request #842 from python-discord/bug/util/831/snekbox-reeval-parsing | 2020-03-26 | -8/+59 | ||
| |\ \ \ \ | |_|/ / |/| | | | Fix snekbox's message parsing to extract code for re-evaluation | ||||
| | * | | | Merge branch 'master' into bug/util/831/snekbox-reeval-parsing | 2020-03-26 | -2/+2 | ||
| | |\ \ \ | |/ / / |/| | | | |||||
| * | | | | Change Alias warnings to info | 2020-03-26 | -2/+2 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Stuff like "{name} tried to run {command}" and "{command} could not be found" was set as a warning, and so Sentry issues were being created for these. Our rule of thumb is that only actionable things should be warnings. Changed these to Info logs. | ||||