| Commit message (Collapse) | Author | Age | Lines |
| |
|
|
|
|
| |
The channel as moved out of this category. Delete the constant too
since it isn't used anywhere else. Keep the excluded channels a tuple
to conveniently support excluding multiple channels in the future.
|
| |
|
|
|
| |
The import was an outlier compared to how the other modules were
imported. It's nicer to keep the imports consistent.
|
| |
|
|
|
| |
Some need to be shared among modules, so it became redundant to redefine
them in each module.
|
| | |
|
| |
|
|
|
|
|
| |
The event is redundant since awaiting the task accomplishes the same
thing. If the task is already done, the await will finish immediately.
If the task gets cancelled, the error is raised but discord.py suppress
it in both commands and event listeners.
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
Plural names imply the modules contain homogenous content. For example,
"channels" implies the module contains multiple kinds of channels.
|
| | |
|
| | |
|
| |
|
|
|
| |
Emoji are no longer used in channel names due to harsher rate limits
for renaming channels. Therefore, the function is obsolete.
|
| | |
|
| |\
| |
| | |
Updated available help channel embed
|
| | |\
| |/
|/| |
|
| |\ \
| | |
| | | |
Include Unicode emojis to emojis filter
|
| | |\ \
| |/ /
|/| | |
|
| | | | |
|
| |\ \ \
| | | |
| | | | |
Fix the deploy stage of our build pipeline
|
| |/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
I've fixed the deploy stage of our build pipeline, as it got mixed in
with the old workflow file due to a merge conflict. The deploy stage is
currently split into a separate workflow; theoretically, this allows us
to trigger a redeploy from GitHub, without having to build the container
image again.
Signed-off-by: Sebastiaan Zeeff <[email protected]>
|
| |\ \ \ |
|
| | |\ \ \
| |/ / /
|/| | | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
python-discord/sebastiaan/backend/improve-actions-workflow
Ensure flake8 actually lints pull requests
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
It's better to document these steps.
Signed-off-by: Sebastiaan Zeeff <[email protected]>
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Sebastiaan Zeeff <[email protected]>
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Sebastiaan Zeeff <[email protected]>
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Unfortunately, our old setup did not actually lint the PR, as it was
running in the context of the target repository. To sidestep the issue
of using `pull_request_target` altogether, I've now changed our run of
flake8 to using it directly and having it output its errors in teh
format of Workflow Commands.
This means that our flake8 output will not be translated automatically
in annotations for the run.
In addition, I've split up the workflow into two separate files: one for
linting & testing and one for building (& deploying).
Signed-off-by: Sebastiaan Zeeff <[email protected]>
|
| |/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We're in the process of migrating snekbox to the GitHub Container
Repository, which will replace DockerHub. I've changed docker-compose to
reflect that change.
Signed-off-by: Sebastiaan Zeeff <[email protected]>
|
| |\ \ \ \ |
|
| | |\ \ \ \
| |/ / / /
|/| | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
It was causing an error if a DSN was not configured. It also feels wrong
and confusing to attempt to make a connection just upon import.
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The classes no longer hold any state since they can use the global bot
instance.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
They're pretty close to being fully static classes, but it's difficult
to make the name attribute a static abstract property.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Is **very** convenient when writing utility functions that rely on the
bot's state, but aren't in cogs and therefore lack the typical way to
access the instance. No more passing around of the instance as an arg!
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | |/ / /
|/| | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Instead detecting only custom emojis, rule now catch too Unicode emojis.
This converts Unicode emojis to :emoji: format and count them.
|
| | |/ /
|/| | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
- This removes a duplicate checkmark from the title of the embed
- The checkmark was left from the previous title system and wasn't removed in the change
|