aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Raise InvalidHeaderError if inventory has invalid headerGravatar Numerlor2021-09-11-7/+24
| | | | | | | | If the requested inventory file has an invalid header, the resource has most likely moved or the url was invalid. In case of adding a new inventory, this error can be handled and displayed to the user. For automatic fetching it'll stop the rescheduling as the request succeeded but returned invalid content.
* Merge pull request #1788 from ↵Gravatar ChrisJL2021-09-09-8/+4
|\ | | | | | | | | python-discord/Only-check-domain-filters-against-URL-like-parts-of-a-message Only check URL-like objects against domain filters
| * Merge branch 'main' into ↵Gravatar ChrisJL2021-09-09-240/+450
| |\ | |/ |/| | | Only-check-domain-filters-against-URL-like-parts-of-a-message
* | Merge pull request #1819 from python-discord/string-formatting-tagGravatar ChrisJL2021-09-09-0/+24
|\ \ | | | | | | Add string-formatting tag
| * \ Merge branch 'main' into string-formatting-tagGravatar ChrisJL2021-09-09-17/+12
| |\ \ | |/ / |/| |
* | | Merge pull request #1813 from python-discord/declare-talentpool-cache-on-startupGravatar ChrisJL2021-09-08-17/+12
|\ \ \ | | | | | | | | Declare TalentPool.cache on init of cog
| * \ \ Merge branch 'main' into declare-talentpool-cache-on-startupGravatar ChrisJL2021-09-08-20/+52
| |\ \ \ | |/ / / |/| | |
| * | | Use - rather than 0 for number of nominees when cache isn't readyGravatar Chris Lovering2021-09-08-1/+1
| | | |
| * | | Remove previous nominations outputGravatar Chris Lovering2021-09-08-11/+0
| | | | | | | | | | | | | | | | This raised questions from helpers when they saw that someone had many previous nominations. There is no reason why a helper needs to see this information.
| * | | Wait until login before trying to use the bot api clientGravatar Chris Lovering2021-09-08-1/+5
| | | |
| * | | Declare and refresh TalentPool.cache on init of cogGravatar Chris Lovering2021-09-06-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | This avoids issues in the server cog trying to access it before it's assigned and refreshed. I also migrated to the tasks to `scheduling.create_task()` as the created tasks currently don't have any error handling they can hide errors in development until the task object is destroyed (if that occurs at all) which logs the exception. The scheduling alternative attaches a callback which logs exceptions to prevent this.
| | * | Change formatting of examplesGravatar Janine vN2021-09-09-16/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After some discussion back and forth, I've adjusted how to display the examples and code bock to be of a more REPL-style. Additionally, a filler character for the "Center Me!" string is added to illustrate how exactly in centers it. This commit also adds some small styling changes.
| | * | Add required newline to end of fileGravatar Janine vN2021-09-09-1/+1
| | | |
| | * | Add string-formatting tagGravatar Janine vN2021-09-09-0/+27
| |/ / |/| | | | | | | | Adds a tag to show the string formatting mini language
* | | Add support for `!infraction last` (#1804)Gravatar TizzySaurus2021-09-08-19/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add support for `!infraction last` `!infraction last` will view details of the last infraction the user made. Also changes the `Infraction` converter to use the `expanded` endpoint 1) Added `InvalidInfraction` to `bot.errors` - called when value passed to `Infraction` converter is invalid 2) Improved error messages for when an invalid infraction is passed to `!infraction {infr_id}` 3) Improved some other messages/docstrings etc. Co-authored-by: Xithrius <[email protected]>
* | | Merge pull request #1814 from python-discord/Expand-fitlers-to-.gg-Gravatar Matteo Bertucci2021-09-06-1/+2
|\ \ \ | |/ / |/| | Add .gg/ to invite filter regex
| * | Add .gg/ to invite filter regexGravatar Chris Lovering2021-09-06-1/+2
|/ /
* | Server command now uses correct TalentPool attr (#1810)Gravatar ChrisJL2021-09-04-1/+1
| | | | | | | | This was attr changed when we removed the concept of a user being 'watched' while removing the talentpool.
* | Merge pull request #1811 from python-discord/pin-platform-in-DockerfileGravatar Johannes Christ2021-09-04-1/+1
|\ \ | | | | | | Pin platform in Dockerfile
| * \ Merge branch 'main' into pin-platform-in-DockerfileGravatar Johannes Christ2021-09-04-0/+19
| |\ \ | |/ / |/| |
* | | Merge pull request #1794 from python-discord/Add-concurrency-to-all-workflowsGravatar ChrisJL2021-09-04-0/+19
|\ \ \ | | | | | | | | Add concurrency rules to all GitHub workflows
| * \ \ Merge branch 'main' into Add-concurrency-to-all-workflowsGravatar ChrisJL2021-09-04-183/+284
| |\ \ \ | |/ / / |/| | |
| * | | Add concurrency rules to all GitHub workflowsGravatar Chris Lovering2021-08-30-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This concurrency rule means that workflow actions will be cancelled if a new set of actions are triggered on the same ref. A ref is either a PR or a branch itself. Doing this means that if someone pushes to a PR multiple times in quick succession, workflows won't get queued up. Instead, only the workflows from the most recent changes will run, as this is all we really care about anyway. I see the benefits of this as twofold. 1. The author gets faster feedback about the most recent change pushed, rather than have to wait for all previous runs to completed 2. Other contributors don't need to wait for a queue of redundant workflows to finish.
| | * | Pin platform in DockerfileGravatar Chris Lovering2021-09-04-1/+1
| |/ / |/| | | | | | | | | | | | | | Some of our deps don't have wheels for atm processors. With Mac's M1 chips becomming more common, we should make it easier for those users to build our environments.
* | | Merge pull request #1808 from python-discord/jb3/update-manifest-pathGravatar Joe Banks2021-09-04-1/+1
|\ \ \ | | | | | | | | Update deployment manifest path
| * | | Update in accordance with python-discord/kubernetes#95Gravatar Joe Banks2021-09-04-1/+1
|/ / /
* | | Merge pull request #1487 from python-discord/ks123/goodbye-talentpool-channelGravatar Xithrius2021-09-03-86/+111
|\ \ \ | | | | | | | | Goodbye, #talent-pool
| * \ \ Merge branch 'main' into ks123/goodbye-talentpool-channelGravatar Xithrius2021-09-03-94/+168
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #1796 from Numerlor/doc-default-base_urlGravatar ChrisJL2021-09-02-3/+12
|\ \ \ \ | | | | | | | | | | Make the base_url argument in doc set optional
| * \ \ \ Merge branch 'main' into doc-default-base_urlGravatar ChrisJL2021-09-02-91/+156
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Add support for searching infractions by infraction id (#1787)Gravatar TizzySaurus2021-09-01-4/+21
| | | | | | | | | | | | | | | | | | | | | | | | | * Add support for searching infractions by infraction id Can now search by infraction id via `!infraction {id}`.
* | | | | Rewording botvars.md tag (#1786)Gravatar Hunter28072021-08-31-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | * Reword botvar.md Co-authored-by: Steele Farnsworth <[email protected]>
* | | | | Merge pull request #1779 from python-discord/experiments/akarys/check-licensesGravatar ChrisJL2021-08-31-86/+134
|\ \ \ \ \ | | | | | | | | | | | | CI: check for dependency licenses
| * \ \ \ \ Merge branch 'main' into experiments/akarys/check-licensesGravatar ChrisJL2021-08-31-37/+78
| |\ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Merge branch 'main' into experiments/akarys/check-licensesGravatar Matteo Bertucci2021-08-28-6/+80
| |\ \ \ \ \
| * | | | | | CI: add ISC License (ISCL) to allowed licensesGravatar Matteo Bertucci2021-08-28-0/+1
| | | | | | |
| * | | | | | Upgrade pip-licenses to 5.3.2Gravatar Matteo Bertucci2021-08-28-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Thanks to https://github.com/raimon49/pip-licenses/pull/109, we are now able to ignore spaces around the allow-only parameter. Rejoice!
| * | | | | | CI: check for dependency licensesGravatar Matteo Bertucci2021-08-25-86/+133
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Since our project is licensed under the MIT License, we can't be using any dependencies in our project. This commit adds a step to the CI lint phase that will use pip-licenses to verify that all the installed packages are part of an ALLOWED_LICENSE variable. This variable is currently set to be every license we currently use. We opted to use an allowlist instead of a denylist to make sure that new licenses are reviewed by hand and added to that variable.
| | | * | | | Make the base_url argument in doc set optionalGravatar Numerlor2021-08-30-3/+12
| |_|/ / / / |/| | | | |
| | | * | | Pop user from talent pool cache when unnominatedGravatar Chris Lovering2021-09-03-0/+1
| | | | | |
| | | * | | `Unnominate reason` to `Unnomination reason`Gravatar Xithrius2021-09-03-1/+1
| | | | | |
| | | * | | Add an extra alias to unnominateGravatar Chris Lovering2021-09-01-1/+1
| | | | | |
| | | * | | Fix spelling of a TalentPool command nameGravatar Chris Lovering2021-08-30-1/+1
| | | | | |
| | | * | | Rename list nominated user command for clarityGravatar Chris Lovering2021-08-30-2/+2
| | | | | |
| | | * | | use global logger in talent pool, over a self var that has been deletedGravatar Chris Lovering2021-08-30-1/+1
| | | | | |
| | | * | | Add back missing default params for TalentPool cogGravatar Chris Lovering2021-08-30-0/+1
| | | | | |
| | | * | | Update unnominate message to remove reference to relaying messages.Gravatar Chris Lovering2021-08-30-1/+1
| | | | | |
| | | * | | Update api endpoint name in talent pool reviewGravatar Chris Lovering2021-08-30-1/+1
| | | | | |
| | | * | | Merge branch 'main' into ks123/goodbye-talentpool-channelGravatar Chris Lovering2021-08-30-2/+4
| | | |\ \ \ | |_|_|/ / / |/| | | | |
* | | | | | Merge pull request #1791 from python-discord/dmrelay-mod-channelGravatar Xithrius2021-08-30-2/+4
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | Allow dmrelay to only be used in mod channels.