aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Deseasonify: lock branding commands to moderation rolesGravatar kwzrd2020-03-15-1/+3
|
* Deseasonify: use more descriptive name for `Record` namedtupleGravatar kwzrd2020-03-15-15/+15
|
* Deseasonify: improve logic in `set` commandGravatar kwzrd2020-03-14-14/+15
| | | | | | | | Prevent re-applying branding if the user requests an already active season. This saves us an expensive operation. Only trigger typing if season switch was successful and `apply` will be called, as otherwise the bot will respond immediately.
* Deseasonify: implement `BrandingManager` cogGravatar kwzrd2020-03-14-0/+312
| | | | | | | | | | | Provides a background daemon capable of autonomously managing the guild's and bot's branding across seasons. Alternatively, the cog can be manually controlled via the `branding` command group. See docstrings for detailed implementation details. Actual calls to Discord API are commented out and replaced with logs, to prevent API abuse during development.
* Deseasonify: improve documentation for `icon_cycle_frequency`Gravatar kwzrd2020-03-14-1/+1
|
* Deseasonify: add `set_banner` method to SeasonalBotGravatar kwzrd2020-03-14-0/+18
|
* Deseasonify: add convenience method to get season by nameGravatar kwzrd2020-03-14-2/+17
|
* Deseasonify: add convenience method to get current seasonGravatar kwzrd2020-03-14-2/+22
|
* Deseasonify: lock daily tasks to their seasons' respective monthsGravatar kwzrd2020-03-08-8/+15
| | | | | | | Applies to daily egg facts and pride facts of the day. Currently, we do not have any sophisticated mechanism in place. However, we can simply configure `active_months` for the cog, and then have the task perform a membership check in each cycle.
* Deseasonify: adjust inheriting seasons in accordance with new systemGravatar kwzrd2020-03-08-66/+26
| | | | | Ensure that all inheriting seasons are now providing the right attrs as defined in `SeasonBase`.
* Deseasonify: remove `Evergreen` seasonGravatar kwzrd2020-03-08-17/+0
| | | | | The `SeasonBase` now serves as the fallback, off-season season for when no other season is available.
* Deseasonify: implement `SeasonBase` for new seasonal systemGravatar kwzrd2020-03-08-12/+16
| | | | See class docstring for implementation details.
* Deseasonify: allow `get_extensions` to find cogs in `seasons`Gravatar kwzrd2020-03-08-3/+7
| | | | | | | | | Add a conditional check to handle cases where `package` is, in fact, just a plain module, and should be loaded directly rather than being searched for modules. The naming here could be improved: `package` does not necessarily have to be a package.
* Deseasonify: remove redundant importGravatar kwzrd2020-03-07-2/+0
|
* Deseasonify: remove empty season moduleGravatar kwzrd2020-03-07-0/+0
| | | | The file snuck back in during merge conflict resolve.
* Deseasonify: create daily tasks inside cog init; wait until readyGravatar kwzrd2020-03-07-2/+6
| | | | | | | | | | Previously, the cog was being instantiated once: once to launch the task, and once to be registered under `bot`. Now, the cog is only created once, and the task is built inside the cog's init. Since cogs are now loaded before the bot connects, the tasks need to wait before the bot's cache is ready, to ensure that the channel is found.
* Deseasonify: remove reference to non-existent task, redundant attrGravatar kwzrd2020-03-07-7/+0
| | | | | | | | | | The `_fact_publisher_task` task was removed many commits ago, yet the listener still attempts to launch it. There is no need to set the channel instance attr, as it isn't used anywhere. This renders the listener useless, and it can be removed.
* Merge branch 'master' into seasonal-purgeGravatar kwzrd2020-03-07-450/+904
|\ | | | | | | | | Update the feature branch. This resolves a conflict in the season.py module, which has been since been deprecated and removed.
| * Merge pull request #370 from python-discord/feat/deps/355/pep8-namingGravatar Sebastiaan Zeeff2020-03-05-0/+1
| |\ | | | | | | Name the use Python version task
| | * CI: name the use Python version taskGravatar MarkKoz2020-03-05-0/+1
| |/ | | | | | | The name is needed to be able to retrieve its output variable.
| * Merge pull request #368 from python-discord/feat/deps/355/pep8-namingGravatar Shirayuki Nekomata2020-03-05-434/+493
| |\ | | | | | | Add pep8-naming and more pre-commit hooks
| | * CI: run pre-commit hooks in CIGravatar MarkKoz2020-03-04-44/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pre-commit creates its own environment in which it installs hooks. To speed up runs, the pipeline will cache this for use with future jobs. The cache will update if .pre-commit-config.yaml changes. The flake8 pre-commit hook invokes flake8 via `pipenv run flake8`. It's normally useful to use pipenv here cause it ensures flake8 is invoked within the context of the venv. However, in CI, there is no venv - dependencies are installed directly to the system site-packages. `pipenv run` does not work in such case because it tries to create a new venv if one doesn't exist (it doesn't consider the system interpreter to be a venv). This workaround (okay, it's a hack) creates an executable shell script which replaces the original pipenv binary. The shell script simply ignores the first argument (i.e. ignores `run` in `pipenv run`) and executes the rest of the arguments as a command. It essentially makes `pipenv run flake8` equivalent to just having ran `flake8`. When pre-commit executes pipenv, the aforementioned script is what will run. * Re-format indentation of azure-pipelines.yml
| | * Add more pre-commit hooksGravatar MarkKoz2020-03-04-100/+113
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Hooks added: * check-merge-conflict - checks for files with merge conflict strings * check-toml - attempts to load all toml files to verify syntax * check-yaml - attempts to load all yaml files to verify syntax * end-of-file-fixer - ensures files end in a newline and only a newline * mixed-line-ending - replaces mixed line endings with LF * trailing-whitespace - trims trailing whitespace * python-check-blanket-noqa - enforces that noqa annotations always occur with specific codes Changes made to comply with new hooks: * Remove trailing whitespaces * Convert some CRLF files to LF * Remove noqa and add missing type annotations * Add missing newlines at end of files See: python-discord/organisation#138
| | * Update dependenciesGravatar MarkKoz2020-03-04-290/+317
| |/ | | | | | | | | | | | | | | | | | | | | | | | | The pipfile will need to be re-locked to add a dependency so may as well re-pin some dependencies to reflect the more recent versions that will be used. discord.py was pinned to a patch version instead of a minor version to be on the safe side. Notable updates: * discord.py -> 1.3.2 * flake8-annotation -> 2.0 * pre-commit -> 2.1
| * Merge pull request #361 from ks129/games-commandGravatar Shirayuki Nekomata2020-03-05-0/+396
| |\ | | | | | | Added .games command
| | * Merge branch 'master' into games-commandGravatar Shirayuki Nekomata2020-03-05-18/+27
| | |\ | | |/ | |/|
| * | Merge pull request #366 from python-discord/tidy-seasonal-channelsGravatar Eivind Teig2020-03-03-15/+12
| |\ \ | | | | | | | | Strip references to seasonalbot chat
| | * | Strip references to seasonalbot chatGravatar S. Co12020-03-01-15/+12
| |/ / | | | | | | | | | Redirect any output to seasonalbot commands, where relevant
| * | Update devlog channel constantGravatar S. Co12020-02-25-1/+1
| | | | | | | | | The log channels have become one
| * | Merge pull request #356 from python-discord/icon-switch-frequencyGravatar Daniel Brown2020-02-25-2/+14
| |\ \ | | | | | | | | Icon switch frequency
| | * | Cycle icons within season in configured interval onlyGravatar kwzrd2020-02-23-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `load_seasons` task now has an internal state, which increments for every day. Before cycling the icon within the same season, first check whether we've waited the configured amount of days since the last cycle. Entering a new season, or changing the icon, resets the state to 0. This allows us to slow down the rate at which we cycle icons, addressing a bug where the icon wasn't loading for some users.
| | * | Add icon cycle frequency constantGravatar kwzrd2020-02-23-0/+1
| | | |
| | * | Calculate seconds as n_hours * seconds_in_hourGravatar kwzrd2020-02-23-1/+1
| |/ / | | | | | | | | | This is a lot more readable than just 86400.
| | * (Games Cog): Added space between game search result + removed cutoff in ↵Gravatar Karlis S2020-03-04-2/+2
| | | | | | | | | | | | get_close_matches.
| | * (Games Cog): Moved `string.Template` to `str.format()`, applied changes ↵Gravatar ks1232020-03-04-51/+37
| | | | | | | | | | | | everywhere.
| | * (Games Cog): Moved self.genres to __init__ and added type hints. Added lower ↵Gravatar ks1232020-03-04-3/+3
| | | | | | | | | | | | `difflib.get_close_matches` cutoff from 0.6 (default) to 0.4.
| | * (Games Cog): Replaced - with space in genre aliases, added multiword genres ↵Gravatar ks1232020-03-04-9/+10
| | | | | | | | | | | | support for .games command, modified docstring to explain this and added str.title() to embed title genre showing.
| | * (Games Cog): Remove too much empty lines in .games search command, simplify ↵Gravatar ks1232020-03-04-1/+1
| | | | | | | | | | | | lines.
| | * (Games Cog): Stop refreshing genres task when Cog unloadGravatar ks1232020-03-04-0/+5
| | |
| | * (Games Cog): Added try block to genres refresh task.Gravatar ks1232020-03-04-1/+6
| | |
| | * (Games Cog): Added .games refresh|r command for refreshing genres.Gravatar ks1232020-03-04-1/+13
| | |
| | * (Games Cog): Created task for fetching genres (every hour)Gravatar ks1232020-03-04-2/+7
| | |
| | * (Games Cog): Fixed _get_genres function looping over genres (started using ↵Gravatar ks1232020-03-04-5/+5
| | | | | | | | | | | | dict.items())
| | * (Games Cog): Fixed get_games_list calling formatting at L192Gravatar ks1232020-03-04-2/+1
| | |
| | * (Games Cog): Fixed companies list generating code (.games <genre> command).Gravatar ks1232020-03-02-3/+2
| | |
| | * (Games Cog): Added comments about offsets, use keyword parameters for ↵Gravatar ks1232020-03-02-1/+5
| | | | | | | | | | | | get_companies_list.
| | * (Games Cog): Fixed and added content to docstrings.Gravatar ks1232020-03-02-2/+7
| | |
| | * (Games Cog): Moved layouts, request bodies and URLs to Templates. Added ↵Gravatar ks1232020-02-28-162/+213
| | | | | | | | | | | | token check on load. Other small code improvisations.
| | * Added .games command with all it's subcommands, added IGDB token requirement ↵Gravatar Karlis S2020-02-25-19/+4
| | | | | | | | | | | | to constants.py.
| | * Remove keys() from total_rating count (Games Cog)Gravatar Karlis S2020-02-25-1/+1
| | | | | | | | | Co-Authored-By: Thomas Petersson <[email protected]>