| Commit message (Collapse) | Author | Age | Lines | ||
|---|---|---|---|---|---|
| ... | |||||
| | | | | * | | | | | | Handle broad exceptions. | 2020-05-07 | -0/+7 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the case an unexpected exception happens, this allows us to try the request again or let the function exit gracefully in the case of multiple fails. | ||||
| | | | | * | | | | | | Continue on failed connections. | 2020-05-07 | -0/+1 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Not using skipping the iteration but continuing directly caused `response_json` being checked but not defined in case of connection errors. Co-authored-by: MarkKoz <[email protected]> | ||||
| | | | | * | | | | | | Use new util function for uploading output. | 2020-05-06 | -11/+2 | ||
| | | | | | | | | | | | |||||
| | | | | * | | | | | | Handle failed paste uploads. | 2020-05-06 | -1/+6 | ||
| | | | | | | | | | | | |||||
| | | | | * | | | | | | Increase log level. | 2020-05-06 | -1/+1 | ||
| | | | | | | | | | | | |||||
| | | | | * | | | | | | Send long eval output to paste service. | 2020-05-06 | -0/+10 | ||
| | | | | | | | | | | | |||||
| | | | | * | | | | | | Attempt requests multiple times with connection errors. | 2020-05-05 | -12/+17 | ||
| | | | | | | | | | | | |||||
| | | | | * | | | | | | Create utility function for uploading to paste service. | 2020-05-05 | -0/+33 | ||
| | | | | | | | | | | | |||||
| | | | | | | | * | | | Categorise most of the uncategorised extensions | 2020-08-19 | -5/+5 | ||
| | | | | | | | | | | | |||||
| | | | | | | | * | | | Dynamically discover and load extensions upon startup | 2020-08-19 | -61/+9 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Being explicit is nice, but the list of extensions to load has gotten quite long. It's a bit of an eyesore. It's still fairly easy to temporarily exclude extensions: just remove them from the set. Granted, being able to comment them out was more convenient. | ||||
| | | | | | | | * | | | Extensions: move utility functions to a utility module | 2020-08-19 | -29/+35 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Makes the cog cleaner and makes the functions more accessible for other modules. | ||||
| | | | | | | | * | | | Extensions: beautify name unqualification | 2020-08-19 | -3/+7 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Yes, that's a real word. | ||||
| | | | | | | | * | | | Replace relative imports with absolute ones | 2020-08-19 | -13/+13 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PEP 8 recommends absolute imports over relative ones. | ||||
| | | | | | | | * | | | Add comment to explain why import is deferred | 2020-08-19 | -0/+1 | ||
| | | | | | | | | | | | |||||
| | | | | | | | * | | | Rename "cogs" directory to "exts" | 2020-08-14 | -163/+164 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The directory contains modules, which are extensions. It only indirectly contains cogs through the extensions. Therefore, a technically more accurate name is "extensions", or "exts" when abbreviated. Furthermore, "exts" is consistent with SeasonalBot. | ||||
| | | | | | | | * | | | Defer imports in extensions using __init__.py | 2020-08-14 | -1/+1 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since `pkgutil.walk_packages` imports packages it comes across, it's best to avoid potential side effects from imports. | ||||
| | | | | | | | * | | | Extensions: support unqualified extension names | 2020-08-14 | -3/+18 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's convenient for users to type less to specify the exception they want. Only require a qualified name if an unqualified name is ambiguous (i.e. two modules in different subpackages have identical names). | ||||
| | | | | | | | * | | | Extensions: refactor category grouping code | 2020-08-14 | -3/+2 | ||
| | | | | | | | | | | | |||||
| | | | | | | | * | | | Extensions: support nested groupings in list command | 2020-08-14 | -6/+5 | ||
| | | | | | | | | | | | |||||
| | | | | | | | * | | | Extensions: group by category in list command | 2020-08-14 | -9/+29 | ||
| | | | | | | | | | | | |||||
| | | | | | | | * | | | Extensions: adjust discovery to work with dir structure | 2020-08-14 | -8/+28 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Discover extensions recursively and ignore any modules/packages whose names start with an underscore. | ||||
| | | | | | | | * | | | Fix paths used to load extensions | 2020-08-14 | -38/+55 | ||
| | | | | | | | | | | | |||||
| | | | | | | | * | | | Fix ModLog imports | 2020-08-14 | -8/+8 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bunch of modules still rely on importing the cog directly from the moderation package. | ||||
| | | | | | | | * | | | Prefix names of non-extension modules with _ | 2020-08-14 | -87/+100 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This naming scheme will make them easy to distinguish from extensions. | ||||
| | | | | | | | * | | | Restructure tests and fix broken tests | 2020-08-14 | -37/+37 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The cog tests structure should mirror the structure of the cogs folder. Fix some import/patch paths which broke due to the restructure. | ||||
| | | | | | | | * | | | Move extensions into sub-directories | 2020-08-13 | -23/+23 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Apply recent changes of notify infraction to test | 2020-09-20 | -3/+2 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Try to fix location from where post infraction test get ID | 2020-09-20 | -2/+2 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Remove useless textwrap import | 2020-09-20 | -1/+0 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Remove special shortening from reason | 2020-09-20 | -1/+1 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Shorten infraction text when any other field than reason is too long | 2020-09-13 | -0/+4 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Restore newlines for `notify_infraction` embed description | 2020-07-06 | -14/+14 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Truncate reason instead full content to avoid removing newlines | ||||
| | | | | | | | | * | | Remove unnecessary if statement from send_private_embed test | 2020-06-25 | -2/+1 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Reorder tests order to match with original file | 2020-06-24 | -65/+65 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Remove unnecessary `textwrap.dedent` in `notify_infraction` | 2020-06-24 | -6/+4 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Move infraction description template back to main file, apply it there | 2020-06-24 | -16/+18 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Fix test case key name in `test_notify_infraction` | 2020-06-24 | -1/+1 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Add other parameters to `test_post_user` `not_user` mock | 2020-06-24 | -5/+5 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Remove unnecessary `log.debug` assert in `test_post_user` | 2020-06-24 | -8/+1 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Merge 2 if-else branches is `test_post_user` | 2020-06-24 | -5/+2 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Replace `True` test with `In` test on `test_post_user` | 2020-06-24 | -1/+1 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Sync keys, variable names and kwargs in `test_post_user` | 2020-06-24 | -12/+12 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Remove case variable definitions in `test_notify_infraction` | 2020-06-24 | -10/+6 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Mod Utils Tests: Remove unnecessary `user` from test name | 2020-06-24 | -1/+1 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Mod Utils Tests: Change `True` assert to `In` assert for message check | 2020-06-24 | -2/+3 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Mod Utils Tests: Move some test cases to `namedtuple` | 2020-06-24 | -66/+29 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Mod Utils Tests: Move INFRACTION_DESCRIPTION_TEMPLATE to tests file | 2020-06-24 | -11/+11 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Mod Utils: Remove unnecessary line splitting on embed footer adding | 2020-06-24 | -3/+1 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Increase coverage of moderation utils tests | 2020-06-12 | -5/+36 | ||
| | | | | | | | | | | | |||||
| | | | | | | | | * | | Implement description shortening to infraction notify tests | 2020-06-12 | -8/+27 | ||
| | | | | | | | | | | | |||||