| Commit message (Collapse) | Author | Age | Lines | |
|---|---|---|---|---|
| * | Boolean Logic fix | 2020-02-22 | -1/+1 | |
| | | | | | | | If we sent a message, we need to return True, the error_handler (in case it called the func) will not send another message if already sent one. | |||
| * | Nontype handling Logic fix | 2020-02-22 | -1/+1 | |
| | | | | | we cannot perform anything without the tags cog and commands | |||
| * | Merge remote-tracking branch 'origin/master' into feat/F4zi/CommandSuggestion | 2020-02-22 | -444/+783 | |
| |\ | ||||
| | * | Change defcon log levels from warning to info | 2020-02-21 | -2/+2 | |
| | | | ||||
| | * | Enable logging warnings to Sentry | 2020-02-21 | -3/+12 | |
| | | | ||||
| | * | Merge pull request #733 from kwzrd/unittest-antispam-rules | 2020-02-21 | -178/+634 | |
| | |\ | | | | | | | Add test cases for antispam rules, fix bug in attachments rule & test | |||
| | | * | Merge branch 'master' into unittest-antispam-rules | 2020-02-21 | -658/+494 | |
| | | |\ | | |/ | |/| | ||||
| | * | | Merge pull request #762 from python-discord/setup-sentry-sdk | 2020-02-21 | -145/+40 | |
| | |\ \ | | | | | | | | | Sentry SDK setup | |||
| | | * | | Attach jump_to url to the event | 2020-02-21 | -0/+8 | |
| | | | | | ||||
| | | * | | Use push_scope instead of configure_scope | 2020-02-21 | -2/+2 | |
| | | | | | ||||
| | | * | | Use log.error instead of capture_exception | 2020-02-21 | -2/+2 | |
| | | | | | ||||
| | | * | | Remove AIOHTTP integration from Sentry since it is intended for AIOHTTP servers. | 2020-02-21 | -3/+1 | |
| | | | | | ||||
| | | * | | Remove tests for custom bot log | 2020-02-21 | -62/+2 | |
| | | | | | ||||
| | | * | | Add basic sentry usage for command errors giving user information and ↵ | 2020-02-21 | -85/+10 | |
| | | | | | | | | | | | | | | | | | command context. | |||
| | | * | | Attach extra information on a command error | 2020-02-21 | -1/+13 | |
| | | | | | ||||
| | | * | | Set BOT_SENTRY_DSN environment variable for tests | 2020-02-21 | -1/+1 | |
| | | | | | ||||
| | | * | | Implement basic Sentry SDK usage | 2020-02-21 | -2/+14 | |
| | |/ / | ||||
| | * | | Merge pull request #761 from python-discord/logging | 2020-02-20 | -124/+81 | |
| | |\ \ | | | | | | | | | Refactor logging setup to be more explicit and cleaner. | |||
| | | * | | Update flake8-annotations to v2.0. | 2020-02-20 | -10/+51 | |
| | | | | | ||||
| | | * | | Change to explicit logs, remove logmatic. | 2020-02-20 | -160/+76 | |
| | |/ / | ||||
| | | * | Add unit test for newlines antispam rule | 2020-02-04 | -0/+105 | |
| | | | | ||||
| | | * | Add unit test for duplicates antispam rule | 2020-02-04 | -0/+66 | |
| | | | | ||||
| | | * | Merge branch 'master' into unittest-antispam-rules | 2020-02-02 | -1/+10 | |
| | | |\ | ||||
| | | * | | Make RuleTest use ABCMeta | 2020-02-02 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | This will prevent child classes to be instantiated unless they implement all abstract methods, leading to a more descriptive error message. | |||
| | | * | | Adjust existing tests to inherit from RuleTest ABC | 2020-02-02 | -294/+157 | |
| | | | | | ||||
| | | * | | Implement RuleTest ABC | 2020-02-02 | -0/+76 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will serve as an ABC for tests for individual rules. The base class provides runners for allowed and disallowed cases, and the children classes then only provide the cases and implementations of helper methods specific to each rule. | |||
| | | * | | Adjust multi-line docstrings to prevailing style | 2020-01-31 | -3/+6 | |
| | | | | | ||||
| | | * | | Refactor msg helper function name to make_msg | 2020-01-26 | -25/+25 | |
| | | | | | | | | | | | | | | | | | The name msg is less descriptive and creates a needless name conflict in local gen exp. | |||
| | | * | | Fix incorrect config key in attachments antispam rule | 2020-01-26 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | The rule was incorrectly printing out the maximum amount of allowed attachments instead of the configured interval. This commit also adjusts the rule's unit test case. | |||
| | | * | | Add unit test case for role mentions antispam rule | 2020-01-26 | -0/+66 | |
| | | | | | ||||
| | | * | | Add unit test case for discord emojis antispam rule | 2020-01-26 | -0/+68 | |
| | | | | | ||||
| | | * | | Add unit test case for chars antispam rule | 2020-01-26 | -0/+75 | |
| | | | | | ||||
| | | * | | Add unit test case for burst shared antispam rule | 2020-01-26 | -0/+65 | |
| | | | | | ||||
| | | * | | Add unit test case for burst antispam rule | 2020-01-26 | -0/+69 | |
| | | | | | ||||
| * | | | | (display_tag) Return False if no tags were found. | 2020-02-22 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | Since no tags were found, we can return false since we didn't send any matching tag | |||
| * | | | | Return False (not sent) at the end of display_tag | 2020-02-22 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | If the code ran to the last line of the function without returning (we return when we send a tag) It means we didn't send the tag and need to return False | |||
| * | | | | awaited can_run Coroutine | 2020-02-22 | -1/+1 | |
| | | | | | ||||
| * | | | | Handling CommandError and Optimization | 2020-02-22 | -6/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After CommandError was catched, the embed was sent, now we return after the exception was catched Modified the way raw_commands are collected | |||
| * | | | | Only replace the misspelt command, and not the whole occurrences of it | 2020-02-21 | -1/+1 | |
| | | | | | ||||
| * | | | | Handling and logging CommandError | 2020-02-21 | -10/+14 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the CommandError exception was ignored, We now catch the exception to handle and log the event Also added a little code reformat | |||
| * | | | | Command suggestion message reformat. | 2020-02-20 | -2/+6 | |
| | | | | | | | | | | | | | | | | | The message sent when suggesting a command is now wrapped inside an embed | |||
| * | | | | Rearrange - Function naming and Docstrings | 2020-02-20 | -105/+81 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | get_command was changed to - display_tag, the name didn't fit, since its not the command itself. command_on_cooldown was taken out of display_tag to get the option to reuse it in another scope Docstrings modified | |||
| * | | | | Rearrange - Function naming and Docstrings | 2020-02-20 | -1/+1 | |
| | | | | | ||||
| * | | | | Return Logic Fixed | 2020-02-19 | -2/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The returned value was changed from True to False If we didn't return True (sent the tag) throughout the code It means we didn't send anything, we should return False | |||
| * | | | | Rearrange, NoneTypes handling | 2020-02-19 | -3/+3 | |
| | | | | | ||||
| * | | | | Rearrange, Function naming and Docstrings | 2020-02-19 | -5/+16 | |
| | | | | | ||||
| * | | | | Rearrange, exception and NoneTypes handling | 2020-02-19 | -12/+16 | |
| | | | | | ||||
| * | | | | Removal of unused logs | 2020-02-19 | -3/+0 | |
| | | | | | ||||
| * | | | | Suggest aliases when invoked instead of full command name | 2020-02-19 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | When found a similar command, the bot sent a suggestion about the full command name This was fixed by saving the matching alias and sending it instead of sending the full command name | |||
| * | | | | Delete_after added for suggestion message | 2020-02-19 | -1/+2 | |
| | | | | | | | | | | | | | | | | | preventing message spam by the bot when commands are misspelt | |||
