|  | Commit message (Collapse) | Author | Age | Lines | 
|---|
| ... |  | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | In the case we receive `"message"` in the json response,
the server had an internal error and we can attempt the request again. | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 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. | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 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]> | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 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. | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | Makes the cog cleaner and makes the functions more accessible for other
modules. | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | Yes, that's a real word. | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | PEP 8 recommends absolute imports over relative ones. | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 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. | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | Since `pkgutil.walk_packages` imports packages it comes across, it's
best to avoid potential side effects from imports. | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 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). | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | Discover extensions recursively and ignore any modules/packages whose
names start with an underscore. | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | Bunch of modules still rely on importing the cog directly from the
moderation package. | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | This naming scheme will make them easy to distinguish from extensions. | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | The cog tests structure should mirror the structure of the cogs folder.
Fix some import/patch paths which broke due to the restructure. | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | Truncate reason instead full content to avoid removing newlines | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | |  |