Skip to content

Conversation

@dawnofmidnight
Copy link
Contributor

Both Python and Sir Lancebot had slightly outdated lists for environment variables, which were updated in this PR.

Changes

Python:
Add BOT_TRACE_LOGGERS
Remove REDDIT_CLIENT_ID and REDDIT_SECRET

Sir Lancebot:
Add REDDIT_CLIENT_ID, REDDIT_SECRET and REDDIT_WEBHOOK to the Token/APIs portion of the env var reference.

Preview:

Python:
image

Sir Lancebot:
image

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 0900c9e on dawnofmidnight:contributing-env-vars into 91cc514 on python-discord:main.

Copy link
Member

@ChrisLovering ChrisLovering left a comment

Choose a reason for hiding this comment

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

LGTM! Nice :D

Copy link
Contributor

@ToxicKidz ToxicKidz left a comment

Choose a reason for hiding this comment

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

Yep :shipit:

@ChrisLovering ChrisLovering merged commit 0e5681c into python-discord:main May 14, 2021
@dawnofmidnight dawnofmidnight deleted the contributing-env-vars branch May 14, 2021 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants