| Commit message (Collapse) | Author | Age | Lines | 
| ...            |  | 
| | | | | | | |  | 
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
discord.py yields duplicate Command objects for each alias a command
has, so the duplicates need to be removed on our end.
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
* Rename `walk_extensions` to `walk_modules` because some extensions
  don't consist of a single module
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
Have to check the modules are equal to prevent yielding imported cogs.
 | 
| | | | | | | |  | 
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
This will help reduce nesting in the actual test.
 | 
| | | | | | | |  | 
 | 
| | | | | | | |  | 
 | 
| | | | | | | |  | 
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
For tests, ideally creating instances of cogs should be avoided to avoid
extra code execution. This function was copied over from discord.py
because their function is not a static method, though it still works as
one. It was probably just a design decision on their part to not make it
static.
 | 
| | | |_|_|_|/  
| |/| | | |    | 
 | 
| | | | | | |  | 
 | 
| | | | | | |  | 
 | 
| | |/ / / /  
|/| | | |   
| | | | |   
| | | | |    | 
empty and with this message formatting go wrong.
 | 
| |\ \ \ \ \  
| |/ / / /  
|/| | | |    | 
Removed API dependency for Tags
 | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | |  | 
Fixed `!tag search any` raises `AttributeError`.
Changed default value of `keywords` from `None` to `'any'`.
This will make it search for keyword `'any'` when there is no keyword.
 | 
| | | | | |  | 
 | 
| | |\ \ \ \  
| | | | | | 
| | | | | | 
| | | | | |  | 
tags_overhaul
 | 
| | | | | | |  | 
 | 
| | | | | | | 
| | | | | | 
| | | | | | 
| | | | | | 
| | | | | |  | 
* Add a constant for the footer text
* Import constants module rather than its classes
 | 
| | |/ / / /   | 
 | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | |  | 
The tags have now been shifted from the database to being static files and hence the get_tags()
method has undergone changes. It now dosen't fetch from the database but looks at the local files
and we need not call it more than once.
 | 
| | |\ \ \ \  
| | | | | | 
| | | | | | 
| | | | | |  | 
tags_overhaul
 | 
| | | |\ \ \ \  
| |_|/ / / /  
|/| | | | |    | 
 | 
| | |/ / / /   | 
 | 
| | | | | |  | 
 | 
| | | | | |  | 
 | 
| | | | | |  | 
 | 
| | |\ \ \ \   | 
 | 
| | | | | | | 
| | | | | | 
| | | | | | 
| | | | | |  | 
"get_tags()" and "_get_tag()" functions need not be async as we are no longer doing any API call but instead reading from local files.
 | 
| | | | | | | 
| | | | | | 
| | | | | | 
| | | | | |  | 
The pathlib module simplifies opening and reading files, hence the os module and the context manager are no longer used.
 | 
| | |\ \ \ \ \  
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
tags_overhaul
 | 
| | | |\ \ \ \ \   | 
 | 
| | |/ / / / / /  
| | | | | | |   
| | | | | | |   
| | | | | | |    | 
caching it after the tags command is used.
 | 
| | | | | | | |  | 
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
of points on github
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
access the static tag files rather than sending an API get request. Removed all methods calling the API so the tags cannot be edited, added nor deleted.
 | 
| | | | | | | |  | 
 | 
| | | | | | | |  | 
 | 
| | | | | | | |  | 
 | 
| | | | | |\ \ \  
| |_|_|_|/ / /  
|/| | | | | |    | 
 | 
| | | | | |\ \ \   | 
 | 
| | | | | |\ \ \ \   | 
 | 
| | | | | |\ \ \ \ \   | 
 | 
| | | | | | | | | | |  | 
 | 
| | | | | | | | | | |  | 
 | 
| | | | | | | | | | |  | 
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
- Docstrings for `apply_ban()` have been edited to mention that the method also removes a banned user from the watch list.
- Parameter `banned` in `apply_unwatch()` was changed to `send_message` in order to be more general.  Boolean logic was swapped to coincide with that change.
- `apply_unwatch()`'s sent message moved to the bottom of the method for clarity.  Added `return`s to the method to exit early if no message needs to be sent.
Signed-off-by: Daniel Brown <[email protected]>
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
Co-Authored-By: Mark <[email protected]>
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
Co-Authored-By: Mark <[email protected]>
 |