aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | * | | | | | | 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
* | | | | | | | Finish .set and .get, and add tests.Gravatar Leon Sandøy2020-05-23-25/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The .set and .get will accept ints, floats, and strings. These will be converted into "typestrings", which is basically just a simple format that's been invented for this object. For example, an int looks like `b"i|2423"`. Note how it is still stored as a bytestring (like everything in Redis), but because of this prefix we are able to coerce it into the type we want on the way out of the db.
* | | | | | | | Use autospecced mocks in MockBot for the stats and aiohttp sessionGravatar MarkKoz2020-05-22-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will help catch anything that tries to get/set an attribute/method which doesn't exist. It'll also catch missing/too many parameters being passed to methods.
* | | | | | | | Properly mock the redis pool in MockBotGravatar MarkKoz2020-05-22-9/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because some of the redis pool/connection methods return futures rather than being coroutines, the redis pool had to be mocked using the CustomMockMixin so it could take advantage of `additional_spec_asyncs` to use AsyncMocks for these future-returning methods.
* | | | | | | | Fix unawaited coro warning when instantiating Bot for MockBot's specGravatar MarkKoz2020-05-22-10/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The fix is to mock the loop and pass it to the Bot. It will then set it as `self.loop` rather than trying to get an event loop from asyncio. The `create_task` patch has been moved to this loop mock rather than being done in MockBot to ensure that it applies to anything calling it when instantiating the Bot.
* | | | | | | | Improve aiohttp context manager mocking in snekbox testsGravatar MarkKoz2020-05-22-3/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I'm not sure how it even managed to work before. It was calling the `post` coroutine (without specifying a URL) and then changing `__aenter__`. Now, a separate mock is created for the context manager and the `post` simply returns that mocked context manager.
* | | | | | | | Fix assertion for `create_task` in duck pond testsGravatar MarkKoz2020-05-22-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The assertion wasn't using the assertion method. Furthermore, it was testing a non-existent function `create_loop` rather than `create_task`.
* | | | | | | | Set up async testbedGravatar Leon Sandøy2020-05-22-198/+135
| | | | | | | |
* | | | | | | | Opens a Redis connection in the Bot class.Gravatar Leon Sandøy2020-05-22-3/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This global connection is the one we will be using in RedisCache to power all our commands. This also ensures that connection is closed when the bot starts its shutdown process.
* | | | | | | | Replace redis-py with aioredis.Gravatar Leon Sandøy2020-05-22-60/+104
| | | | | | | |
* | | | | | | | Add the REDIS_PASSWORD environment variableGravatar Leon Sandøy2020-05-22-137/+115
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In production, we will need this password to make a connection to Redis.
* | | | | | | | Add the REDIS_PASSWORD environment variableGravatar Leon Sandøy2020-05-17-6/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In production, we will need this password to make a connection to Redis.
* | | | | | | | Changes discord-py to discord.py in PipfileGravatar Leon Sandøy2020-05-17-7/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `discord-py` package is no longer the official release, and so making this change silences some warnings about deprecation.
* | | | | | | | Merge branch 'master' into redis_persistenceGravatar Leon Sandøy2020-05-17-460/+269
|\| | | | | | |
| * | | | | | | Merge pull request #519 from mathsman5133/help-refactorGravatar Sebastiaan Zeeff2020-05-17-460/+269
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Refactor the !help command.
| | * \ \ \ \ \ \ Merge branch 'master' into help-refactorGravatar Sebastiaan Zeeff2020-05-17-1/+1
| | |\ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | |
| | * | | | | | | fix redirect_output decorator; remove ninja codeGravatar mathsman51332020-05-16-76/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Lots of instance of `for c in ...` or `for a in ...` or `fmt` which are non-descriptive and sometimes cryptic. - Ves suggested running the command in an asyncio task for `@redirect_output`, rather than making a workaround which only applies to the help command. This fixes a fundamental flaw where the redirection message wouldn't be deleted until a further 60sec after the command has finished, which for `!help` could be up to 5min, meaning the invocation message could be sitting there for 6min, not the intended 60sec.
| | * | | | | | | Merge branch 'master' into help-refactorGravatar Leon Sandøy2020-05-15-447/+2127
| | |\ \ \ \ \ \ \ | | | | |_|/ / / / | | | |/| | | | |
| | * | | | | | | Help: lower score cutoff for fuzzy matchGravatar Mark2020-04-01-1/+1
| | | | | | | | |
| | * | | | | | | Fix linting?Gravatar mathsman51332020-03-31-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Not sure why my precommit didn't pick that up...
| | * | | | | | | Merge branch 'master' of https://github.com/python-discord/bot into ↵Gravatar mathsman51332020-03-31-375/+1918
| | |\ \ \ \ \ \ \ | | | | |_|_|_|_|/ | | | |/| | | | | | | | | | | | | | help-refactor
| | * | | | | | | Apply suggestions from review.Gravatar mathsman51332020-03-31-12/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Make exception handling for bin reaction more specific - Channel constants were updated recently - Suggest category names - Tidy up signature formatting - Move score cutoff to 80 to allow a few more matches
| | * | | | | | | Merge branch 'master' of https://github.com/python-discord/bot into ↵Gravatar mathsman51332020-03-12-2207/+4239
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | help-refactor  Conflicts:  bot/cogs/help.py
| | * | | | | | | | Apply suggestions from Mark's code review.Gravatar mathsman51332020-03-10-60/+63
| | | | | | | | | |
| | * | | | | | | | Use the new :trashcan: emoji to delete the help message, as per #625Gravatar mathsman51332020-02-10-4/+5
| | | | | | | | | |
| | * | | | | | | | Merge branch 'master' of https://github.com/python-discord/bot into ↵Gravatar mathsman51332020-02-10-3174/+7258
| | |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | help-refactor  Conflicts:  bot/cogs/help.py
| | * | | | | | | | | Remove trailing commaGravatar mathsman51332019-11-16-1/+1
| | | | | | | | | | |
| | * | | | | | | | | Apply suggestions from reviewGravatar mathsman51332019-11-16-24/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Description was the same as prefix parameter of paginator - Cleanup is redundant pending closure of #514 - Clean/fix couple if statements in help.py
| | * | | | | | | | | Show a maximum of 8 commands per page rather than 5.Gravatar mathsman51332019-11-03-1/+1
| | | | | | | | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-26 15:08:40 +0000