aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Use a defaultdict for item futuresGravatar Numerlor2021-01-15-2/+2
| | | | | | | To be able to set the attribute at the start of the coro we need to be able to access the item's future before we know about all the other items. This also saves us from having to add them all as the queue parser or get_markdown will create the futures for us dynamically
* Prevent an inventory refresh while waiting for item cacheGravatar Numerlor2021-01-14-0/+7
| | | | | | | If an inventory refresh was started while the symbol embed coroutine was suspended, it could cause the parser to try to fetch a non existent future if the markdown was requested after it was cleared but before new inventories were loaded in.
* Raise an error or log a warning if there's a global name conflictGravatar Numerlor2021-01-14-5/+35
| | | | | | | | | | When wrapper uses a global name, which conflicts with a global name from wrapped's module that wrapped uses for its annotations, we run into a situation that can't be solved without changing one of the names, so an error is raised to give this clearer meaning. The check may be erroneous in some edge cases or the objects the conflicting names refer to can be functionally identical, so the error can be turned into a logged warning.
* Ensure footer is actually max 100 charsGravatar Numerlor2021-01-14-2/+3
| | | | | | Shortening the renamed symbols string to 100 chars is not accurate as the footer also contains a string before that, subtracting its length fixes this.
* Set the user_requested attribute at the start of the coroutineGravatar Numerlor2021-01-14-1/+1
| | | | | | | | A context switch may occur when we're waiting for the web page response, during which a clear could be triggered. If the event is not set before that we could end up with the dictionary changing sizes, or if a copy was made, a future that'd never finish as it'd be cleared from the queue and the futures dict
* Reuse the redis key instead of creating a new string for the expires setGravatar Numerlor2021-01-14-2/+2
|
* Change typehint name casing to PascalCaseGravatar Numerlor2021-01-14-8/+9
|
* Keep trakck of the init task and cancel it when the cog is unloadedGravatar Numerlor2021-01-14-1/+2
|
* Simplify control flowGravatar Numerlor2021-01-14-3/+2
| | | | Co-authored-by: MarkKoz <[email protected]>
* Use different task ids for every inventory reschedule attemptsGravatar Numerlor2021-01-14-6/+6
| | | | | | | The scheduler can't keep track of multiple tasks with the same id, and rescheduling the update task using the same id within an already scheduled update task caused the new task to get ignored as the old task only got deleted from the scheduler after it was finished
* Correct return type annotationGravatar Numerlor2021-01-14-1/+1
|
* Turn update_single into a normal functionGravatar Numerlor2021-01-14-3/+3
| | | | The method no longer runs anything asynchronous
* Fix docstringGravatar Numerlor2021-01-11-1/+1
| | | | The decorator works in revers to what the docstring explained
* Return whitespace to its previous stateGravatar Numerlor2021-01-11-1/+1
|
* Make REFRESH_EVENT an instance variableGravatar Numerlor2021-01-11-6/+6
|
* Merge remote-tracking branch 'upstream/master' into doc-impGravatar Numerlor2021-01-11-1234/+646
|\
| * Merge pull request #1350 from python-discord/mbaruh/developerectomyGravatar Dennis Pham2021-01-10-720/+14
| |\ | | | | | | Annihilate all traces of Developer and Unverified roles
| | * Annihilate all traces of Developer and Unverified rolesGravatar mbaruh2021-01-11-720/+14
| |/
| * Make sure that users without the Developers role can use tag.Gravatar Matteo Bertucci2021-01-10-2/+2
| | | | | | | | | | We have a check in place to restrict tag usage to a certain role, but our default is the Developers role, and some users now don't have this code. This commit fixes this by using None as a default and adding a truth test in the check_accessibility method.
| * Upped duckpond threshold to 5Gravatar Boris Muratov2021-01-08-1/+1
| |
| * Merge pull request #1340 from python-discord/ks123/raw-fixGravatar Matteo Bertucci2021-01-04-2/+6
| |\ | | | | | | Add check does user have permission to get raw message and re-enable raw command
| | * Merge branch 'master' into ks123/raw-fixGravatar Matteo Bertucci2021-01-04-71/+131
| | |\ | | |/ | |/|
| * | Merge pull request #1152 from python-discord/Make_nomination_reason_optionalGravatar Dennis Pham2021-01-03-8/+8
| |\ \ | | | | | | | | Make nomination reason optional.
| | * \ Merge branch 'master' into Make_nomination_reason_optionalGravatar Dennis Pham2021-01-03-45/+60
| | |\ \ | | |/ / | |/| |
| * | | Merge PR #1345 - Update d.py to fix webhook message publishingGravatar Mark2021-01-03-45/+60
| |\ \ \
| | * | | Update discord.py to fix webhook message publishing.Gravatar Den42002021-01-01-45/+60
| |/ / / | | | | | | | | | | | | Related to #1342.
| | * | Removed 'Optional' import.Gravatar xithrius2021-01-02-1/+1
| | | |
| | * | Removed unnecessary debugging logs.Gravatar Xithrius2021-01-02-2/+0
| | | | | | | | | | | | Co-authored-by: Dennis Pham <[email protected]>
| | * | Set reason to default as an empty string.Gravatar Xithrius2021-01-02-5/+2
| | | | | | | | | | | | Co-authored-by: Dennis Pham <[email protected]>
| | * | No nominaton reason blank replaced by italic NoneGravatar xithrius2021-01-02-2/+4
| | | |
| | * | Rebased master into this branch.Gravatar xithrius2021-01-02-6460/+7705
| | |\ \ | | |/ / | |/| | | | | | The cogs folder no longer exists, but the merge command was able to automagically find the correct files to merge into. Nomination reason is now optional.
| * | | Merge pull request #1334 from python-discord/bug/precommit-pycharmGravatar ks1292020-12-31-1/+1
| |\ \ \ | | | | | | | | | | Fix flake8 pre-commit hook running through PyCharm
| | * \ \ Merge branch 'master' into bug/precommit-pycharmGravatar ks1292020-12-31-297/+322
| | |\ \ \ | | |/ / / | |/| | |
| * | | | Merge pull request #1339 from python-discord/swfarnsworth/infoGravatar Dennis Pham2020-12-31-3/+3
| |\ \ \ \ | | | | | | | | | | | | `!user` command says if a user is "Verified", rather than "Pending"
| | * | | | "Pending: False" to "Verified: True" to agree with new semantics.Gravatar Steele2020-12-30-1/+1
| | | | | |
| | * | | | Merge branch 'master' of https://github.com/python-discord/bot into ↵Gravatar Steele2020-12-30-14/+59
| | |\ \ \ \ | | |/ / / / | |/| | | | | | | | | | swfarnsworth/info
| * | | | | Merge pull request #1016 from ks129/bug-fixesGravatar ks1292020-12-29-14/+59
| |\ \ \ \ \ | | | | | | | | | | | | | | Some bug fixes
| | * \ \ \ \ Merge branch 'master' into bug-fixesGravatar ks1292020-12-29-554/+567
| | |\ \ \ \ \ | | |/ / / / / | |/| | | | |
| | * | | | | Log info instead error for watchchannel consume task cancelGravatar ks1292020-12-17-1/+1
| | | | | | |
| | * | | | | Merge branch 'master' into bug-fixesGravatar ks1292020-12-17-55/+274
| | |\ \ \ \ \
| | * \ \ \ \ \ Merge branch 'master' into bug-fixesGravatar ks1292020-12-02-1079/+1215
| | |\ \ \ \ \ \
| | * | | | | | | Remove unnecessary f-stringGravatar ks1292020-12-02-1/+1
| | | | | | | | |
| | * | | | | | | Fix removing extensions and cogs for bot shutdownGravatar ks1292020-12-02-3/+7
| | | | | | | | |
| | * | | | | | | Add failure message when applying infraction fails because user leftGravatar ks1292020-12-02-0/+2
| | | | | | | | |
| | * | | | | | | Remove second unnecessary parenthesisGravatar ks1292020-12-02-4/+2
| | | | | | | | |
| | * | | | | | | Log exception instead warning for unexpected HttpExceptionGravatar ks1292020-12-02-1/+1
| | | | | | | | |
| | * | | | | | | Fix user leaving from guild log grammarGravatar ks1292020-12-02-2/+4
| | | | | | | | |
| | * | | | | | | Merge NotFound caching with HttpException caching with status codeGravatar ks1292020-12-02-10/+4
| | | | | | | | |
| | * | | | | | | Merge branch 'master' into bug-fixesGravatar ks1292020-11-21-604/+931
| | |\ \ \ \ \ \ \
| | * | | | | | | | Catch not found exception in schedulerGravatar ks1292020-11-07-15/+30
| | | | | | | | | |