aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* DRY approach to typestring prefix resolutionGravatar Leon Sandøy2020-05-23-15/+60
| | | | | | | | | | Thanks to @kwzrd for this idea, basically we're making a constant with the typestring prefixes and iterating that in all our converters. These converter functions will also now raise TypeErrors if we try to convert something that isn't in this constants list. I've also added a new test that tests this functionality.
* Don't rely on HDEL ignoring keys for .popGravatar Leon Sandøy2020-05-23-2/+21
| | | | | | | | | | | Previously we would try to .delete keys that did not exist if a default was provided when calling .pop. This is okay to do (because HDEL will just ignore any attempts to delete non-existing calls), but it does add an additional pointless API call to Redis, so I've added some validation as a small optimization. This also adds a few additional lines of documentation as requested by @SebastiaanZ in their review.
* Fix typo in test_to_dict docstringGravatar Leon Sandøy2020-05-23-1/+1
|
* Update exception message Gravatar Leon Sandøy2020-05-23-1/+4
| | | | | | | This was incorrectly suggesting the user needed to create an instance of RedisCache, when in fact it is the parent that needs to be instantiated. Co-authored-by: Sebastiaan Zeeff <[email protected]>
* CI needs REDIS_PASSWORD to pass testsGravatar Leon Sandøy2020-05-23-0/+1
|
* Moving the Redis session creation to Bot._recreateGravatar Leon Sandøy2020-05-23-2/+5
|
* Unbreak the error_handlerGravatar Leon Sandøy2020-05-23-1/+1
|
* Merge branch 'redis_persistence' of github.com:python-discord/bot into ↵Gravatar Leon Sandøy2020-05-23-44/+127
|\ | | | | | | redis_persistence
| * Merge branch 'master' into redis_persistenceGravatar Leon Sandøy2020-05-23-44/+127
| |\
| | * Merge pull request #945 from ks129/statsGravatar Joseph Banks2020-05-22-1/+36
| | |\
| | | * Merge branch 'master' into statsGravatar Joseph Banks2020-05-22-505/+362
| | | |\ | | | |/ | | |/|
| | * | Merge pull request #942 from ks129/python-news-statsGravatar Joseph Banks2020-05-22-0/+6
| | |\ \
| | | * \ Merge branch 'master' into python-news-statsGravatar Mark2020-05-20-505/+356
| | | |\ \ | | | |/ / | | |/| |
| | * | | Merge pull request #944 from Numerlor/eval-timeout-increaseGravatar Dennis Pham2020-05-19-2/+7
| | |\ \ \ | | | | | | | | | | | | Increase snekbox re eval timeout to 30 seconds
| | | * \ \ Merge branch 'master' into eval-timeout-increaseGravatar Dennis Pham2020-05-19-503/+349
| | | |\ \ \ | | | |/ / / | | |/| | |
| | * | | | Add Steam gift card scam to domain blacklistGravatar S. Co12020-05-18-0/+2
| | | | | |
| | * | | | [bug] Adjustment to changes in #941, return message sent by webhook so ↵Gravatar Joseph Banks2020-05-19-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | publish can take place
| | * | | | Merge pull request #941 from ks129/reddit-publishGravatar Dennis Pham2020-05-18-1/+7
| | |\ \ \ \ | | | | | | | | | | | | | | Add message publishing to Reddit cog
| | | * \ \ \ Merge branch 'master' into reddit-publishGravatar Dennis Pham2020-05-18-777/+1183
| | | |\ \ \ \ | | | |/ / / / | | |/| | | |
| | * | | | | Merge pull request #858 from python-discord/decorator-factory-mutability-tagGravatar kwzrd2020-05-17-0/+37
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | Add mutability.md tag
| | | * \ \ \ \ Merge branch 'master' into decorator-factory-mutability-tagGravatar kwzrd2020-05-17-890/+2365
| | | |\ \ \ \ \ | | | |/ / / / / | | |/| | | | |
| | * | | | | | Merge pull request #949 from python-discord/help-command-fix-invocationGravatar kwzrd2020-05-17-41/+33
| | |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Use `send_help` to ensure that our help command is correctly invoked
| | | * | | | | | Use `Command`-object for `send_help`Gravatar Sebastiaan Zeeff2020-05-17-21/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As @mathsman5133 pointed out, it's better to use the `Command`-instance we typically already have in the current context than to rely on parsing the qualified name again. The invocation is now done as: `await ctx.send_help(ctx.command)`
| | | * | | | | | Use `send_help` to invoke command helpGravatar Sebastiaan Zeeff2020-05-17-40/+32
| | |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After the refactoring of the help command, we need to use the built-in method of calling the help command: `Context.send_help`. As an argument, the qualified name (a string containing the full command path, including parents) of the command can be passed. Examples: - await ctx.send_help("reminders edit") This would send a help embed with information on `!reminders edit` to the Context. - await ctx.send_help(ctx.command.qualified_name) This would extract the qualified name of the command, which is the full command path, and send a help embed to Context. - await ctx.send_help() This will send the main "root" help embed to the Context.
| | | * | | | | Fix incomplete variable renamingGravatar decorator-factory2020-05-17-1/+1
| | | | | | | |
| | | * | | | | Rename `string` to `greeting`Gravatar decorator-factory2020-05-17-10/+10
| | | | | | | |
| | | * | | | | Add a note on user-defined classesGravatar decorator-factory2020-05-17-1/+1
| | | | | | | |
| | | * | | | | Change standalone programs to interactive sessionsGravatar decorator-factory2020-05-17-8/+13
| | | | | | | |
| | | * | | | | Apply language improvements proposed from kwzrdGravatar decorator-factory2020-05-17-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: kwzrd <[email protected]>
| | | * | | | | Fix hard-wrapping in mutability.mdGravatar decorator-factory2020-04-01-7/+5
| | | | | | | |
| | | * | | | | header->bold in mutability.mdGravatar decorator-factory2020-04-01-1/+1
| | | | | | | |
| | | * | | | | Add mutability.md tagGravatar decorator-factory2020-04-01-0/+34
| | | | | | | |
| | | | * | | | Add message publishing to `Reddit` cogGravatar ks1292020-05-13-1/+7
| | | | | | | |
| | | | | * | | Change tests to use the new timeout constantGravatar Numerlor2020-05-15-1/+5
| | | | | | | |
| | | | | * | | Move the re eval timeout to a module constantGravatar Numerlor2020-05-15-1/+2
| | | | | | | |
| | | | | * | | Increase snekbox re eval timeout.Gravatar Numerlor2020-05-15-1/+1
| | | | | | | |
| | | | | | * | Merge branch 'master' into python-news-statsGravatar ks1292020-05-14-2/+1
| | | | | | |\ \ | | | | | | |/ / | | | | | |/| |
| | | | | | * | Python News: Implement statsGravatar ks1292020-05-14-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | Add stat increaser to PEP and maillist posting.
| | | | | | | * Eval Stats: Replaced `elif` with `else` on icon checkGravatar ks1292020-05-22-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]>
| | | | | | | * Stats: Fix docstringsGravatar ks1292020-05-19-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Joseph Banks <[email protected]>
| | | | | | | * Stats: Fixed stat namesGravatar ks1292020-05-19-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Joseph Banks <[email protected]>
| | | | | | | * Stats: Added stats for eval channel using (Help/Bot commands/Topical)Gravatar ks1292020-05-16-0/+7
| | | | | | | |
| | | | | | | * Stats: Added stats for eval role uses (Helpers/Developers)Gravatar ks1292020-05-16-0/+5
| | | | | | | |
| | | | | | | * Stats: Added stats for eval successes + failsGravatar ks1292020-05-16-0/+6
| | | | | | | |
| | | | | | | * Stats: Added codeblock correction statsGravatar ks1292020-05-16-0/+2
| | | | | | | |
| | | | | | | * Stats: Create guild boost stat collectionGravatar ks1292020-05-16-1/+16
| | | | | | |/ | | | | | |/| | | | | | | | | | | | | | | Collect Guild boost amount + level and post it to StatsD every hour in task. Added starting to cog `__init__.py` and stopping to `cog_unload`.
* | | | | / | Better docstring for RedisCache.containsGravatar Leon Sandøy2020-05-23-1/+5
|/ / / / / /
* | | | | | Better docstring for RedisCacheGravatar Leon Sandøy2020-05-23-1/+31
| | | | | |
* | | | | | Complete asyncified test suite for RedisCacheGravatar Leon Sandøy2020-05-23-94/+112
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit just alters existing code to work with the new interface, and with async. All tests are passing successfully.
* | | | | | Finish asyncifying RedisCache methodsGravatar Leon Sandøy2020-05-23-16/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - All methods will now do a validation check - Complete interface spec added to class: - .update - .clear - .pop - .to_dict - .length - .contains - .delete - .get - .set