Skip to content

Hard-coded PEP 0 to !pep command #848

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 5 commits into from
Mar 29, 2020
Merged

Hard-coded PEP 0 to !pep command #848

merged 5 commits into from
Mar 29, 2020

Conversation

@ks129
Copy link
Member

@ks129 ks129 commented Mar 28, 2020

Closes #805

@ks129 ks129 requested a review from a team as a code owner March 28, 2020 08:55
sco1
sco1 suggested changes Mar 28, 2020
View reviewed changes
Copy link
Contributor

@sco1 sco1 left a comment

Choose a reason for hiding this comment

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

One adjustment, otherwise works fine.

@ks129 ks129 requested a review from sco1 March 28, 2020 18:28
@MarkKoz MarkKoz added a: information Related to information commands: (doc, help, information, reddit, site, tags) p: 3 - low Low Priority t: bug Something isn't working labels Mar 28, 2020
MarkKoz
MarkKoz requested changes Mar 28, 2020
View reviewed changes
@MarkKoz MarkKoz added the s: waiting for author Waiting for author to address a review or respond to a comment label Mar 28, 2020
@ks129 ks129 requested a review from MarkKoz March 29, 2020 06:09
MarkKoz
MarkKoz approved these changes Mar 29, 2020
View reviewed changes
Copy link
Member

@MarkKoz MarkKoz left a comment

Choose a reason for hiding this comment

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

Thanks!

@sco1 sco1 merged commit 26b87b4 into python-discord:master Mar 29, 2020
@ks129 ks129 deleted the pep0 branch March 29, 2020 16:12
@Xithrius Xithrius removed the s: waiting for author Waiting for author to address a review or respond to a comment label May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MarkKoz MarkKoz MarkKoz approved these changes

+1 more reviewer

@sco1 sco1 sco1 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

a: information Related to information commands: (doc, help, information, reddit, site, tags) p: 3 - low Low Priority t: bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Pep command cannot fetch PEP 0.

4 participants

@ks129 @MarkKoz @sco1 @Xithrius