Skip to content

Ignore what's new for MentionBot#521

Merged
brettcannon merged 2 commits intopython:masterfrom
Carreau:mentionbot-ignore
Mar 6, 2017
Merged

Ignore what's new for MentionBot#521
brettcannon merged 2 commits intopython:masterfrom
Carreau:mentionbot-ignore

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
"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.
@Carreau Carreau deleted the mentionbot-ignore branch October 20, 2025 13:16
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