Skip to content
This repository was archived by the owner on Feb 13, 2022. It is now read-only.

Conversation

@ChrisLovering
Copy link
Member

Update thread bot to use the latest discord.py version from git, since we don't need to worry about pinning this.

This allows us to remove git as a requirement in the Docker image.

Some minor breaking changes were made since our pinned version, so those have been fixed in here too.

@ChrisLovering ChrisLovering mentioned this pull request Aug 28, 2021
Bluenix2
Bluenix2 previously approved these changes Aug 28, 2021
Copy link
Member

@Bluenix2 Bluenix2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simple enough, I used Sourcegraph to search the repository for things mentioned as breaking changes and found nothing. Thanks Chris

Since the repo is archived now, no need to pin to a specific revision. Not doing this will allow us to remove git from the image too.
Since we use an archive url now, rather than git, we no longer need git to be installed.
We no longer get these as raw emojis anymore, but rather the full discord emoji text
@ChrisLovering ChrisLovering force-pushed the Bump-d.py-to-latest-master-branch-changes branch from 303b441 to 0a1c75a Compare August 29, 2021 19:36
@ChrisLovering ChrisLovering merged commit 442ec9c into main Aug 29, 2021
@ChrisLovering ChrisLovering deleted the Bump-d.py-to-latest-master-branch-changes branch August 29, 2021 20:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants