Skip to content

Ignore what's new for MentionBot #521

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.

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

Merged
merged 2 commits into from
Mar 6, 2017

Conversation

Carreau
Copy link
Contributor

@Carreau Carreau commented Mar 6, 2017

Because everybody is editing in every PR and that's finding wrong
reviewers.

Because everybody is editing in every PR and that's finding wrong
reviewers.
@mention-bot
Copy link

@Carreau, thanks for your PR! By analyzing the history of the files in this pull request, we identified @dstufft, @brettcannon and @skrah to be potential reviewers.

@Carreau
Copy link
Contributor Author

Carreau commented Mar 6, 2017

For context I was pinged as a potential reviewer on #505, which I definitively don't have the skills to review, so I assumed it was the what's new.

.mention-bot Outdated
@@ -4,7 +4,8 @@
"findPotentialReviewers": true,
"fileBlacklist": [
"Misc/ACKS",
"Misc/NEWS"
"Misc/NEWS",
"Doc/whatsnew/*.rst",
Copy link
Member

Choose a reason for hiding this comment

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

This is a JSON file and cannot have a trailing comma.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops indeed, fixed.

@brettcannon brettcannon merged commit 0f5f1c3 into python:master Mar 6, 2017
@brettcannon
Copy link
Member

Thanks!

@dstufft
Copy link
Member

dstufft commented Mar 6, 2017

Ah, I guess tests are just failing on master in general not related to this PR.

@brettcannon
Copy link
Member

@dstufft yep, some breakage about keyword arguments; looked like it wasn't related to this

jaraco pushed a commit that referenced this pull request Dec 2, 2022
The current logic is buggy and not worth
the trouble.

Core devs and triagers will be handling this event manually.
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.

5 participants