aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Boolean Logic fixGravatar F4zii2020-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 fixGravatar F4zii2020-02-22-1/+1
| | | | we cannot perform anything without the tags cog and commands
* Merge remote-tracking branch 'origin/master' into feat/F4zi/CommandSuggestionGravatar MarkKoz2020-02-22-444/+783
|\
| * Change defcon log levels from warning to infoGravatar Joseph Banks2020-02-21-2/+2
| |
| * Enable logging warnings to SentryGravatar Joseph Banks2020-02-21-3/+12
| |
| * Merge pull request #733 from kwzrd/unittest-antispam-rulesGravatar Sebastiaan Zeeff2020-02-21-178/+634
| |\ | | | | | | Add test cases for antispam rules, fix bug in attachments rule & test
| | * Merge branch 'master' into unittest-antispam-rulesGravatar Sebastiaan Zeeff2020-02-21-658/+494
| | |\ | | |/ | |/|
| * | Merge pull request #762 from python-discord/setup-sentry-sdkGravatar Mark2020-02-21-145/+40
| |\ \ | | | | | | | | Sentry SDK setup
| | * | Attach jump_to url to the eventGravatar Joseph Banks2020-02-21-0/+8
| | | |
| | * | Use push_scope instead of configure_scopeGravatar Joseph Banks2020-02-21-2/+2
| | | |
| | * | Use log.error instead of capture_exceptionGravatar Joseph Banks2020-02-21-2/+2
| | | |
| | * | Remove AIOHTTP integration from Sentry since it is intended for AIOHTTP servers.Gravatar Joseph Banks2020-02-21-3/+1
| | | |
| | * | Remove tests for custom bot logGravatar Joseph Banks2020-02-21-62/+2
| | | |
| | * | Add basic sentry usage for command errors giving user information and ↵Gravatar Joseph Banks2020-02-21-85/+10
| | | | | | | | | | | | | | | | command context.
| | * | Attach extra information on a command errorGravatar Joseph Banks2020-02-21-1/+13
| | | |
| | * | Set BOT_SENTRY_DSN environment variable for testsGravatar Joseph Banks2020-02-21-1/+1
| | | |
| | * | Implement basic Sentry SDK usageGravatar Joseph Banks2020-02-21-2/+14
| |/ /
| * | Merge pull request #761 from python-discord/loggingGravatar Joseph2020-02-20-124/+81
| |\ \ | | | | | | | | Refactor logging setup to be more explicit and cleaner.
| | * | Update flake8-annotations to v2.0.Gravatar scragly2020-02-20-10/+51
| | | |
| | * | Change to explicit logs, remove logmatic.Gravatar scragly2020-02-20-160/+76
| |/ /
| | * Add unit test for newlines antispam ruleGravatar kwzrd2020-02-04-0/+105
| | |
| | * Add unit test for duplicates antispam ruleGravatar kwzrd2020-02-04-0/+66
| | |
| | * Merge branch 'master' into unittest-antispam-rulesGravatar Joseph2020-02-02-1/+10
| | |\
| | * | Make RuleTest use ABCMetaGravatar kwzrd2020-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 ABCGravatar kwzrd2020-02-02-294/+157
| | | |
| | * | Implement RuleTest ABCGravatar kwzrd2020-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 styleGravatar kwzrd2020-01-31-3/+6
| | | |
| | * | Refactor msg helper function name to make_msgGravatar kwzrd2020-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 ruleGravatar kwzrd2020-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 ruleGravatar kwzrd2020-01-26-0/+66
| | | |
| | * | Add unit test case for discord emojis antispam ruleGravatar kwzrd2020-01-26-0/+68
| | | |
| | * | Add unit test case for chars antispam ruleGravatar kwzrd2020-01-26-0/+75
| | | |
| | * | Add unit test case for burst shared antispam ruleGravatar kwzrd2020-01-26-0/+65
| | | |
| | * | Add unit test case for burst antispam ruleGravatar kwzrd2020-01-26-0/+69
| | | |
* | | | (display_tag) Return False if no tags were found.Gravatar F4zii2020-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_tagGravatar F4zii2020-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 CoroutineGravatar F4zii2020-02-22-1/+1
| | | |
* | | | Handling CommandError and OptimizationGravatar F4zii2020-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 itGravatar F4zii2020-02-21-1/+1
| | | |
* | | | Handling and logging CommandErrorGravatar F4zii2020-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.Gravatar F4zi2020-02-20-2/+6
| | | | | | | | | | | | | | | | The message sent when suggesting a command is now wrapped inside an embed
* | | | Rearrange - Function naming and DocstringsGravatar F4zii2020-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 DocstringsGravatar F4zii2020-02-20-1/+1
| | | |
* | | | Return Logic FixedGravatar F4zii2020-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 handlingGravatar F4zii2020-02-19-3/+3
| | | |
* | | | Rearrange, Function naming and DocstringsGravatar F4zii2020-02-19-5/+16
| | | |
* | | | Rearrange, exception and NoneTypes handlingGravatar F4zii2020-02-19-12/+16
| | | |
* | | | Removal of unused logsGravatar F4zii2020-02-19-3/+0
| | | |
* | | | Suggest aliases when invoked instead of full command nameGravatar F4zii2020-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 messageGravatar F4zii2020-02-19-1/+2
| | | | | | | | | | | | | | | | preventing message spam by the bot when commands are misspelt