If PR is rejected, automatically add Changes Needed label

[awitherow]

Also notify the creator?

• slack channel ?
• comment in PR?

[nicolai86]

@awitherow can you describe in more detail what you mean by "rejected"?

[awitherow]

1. either by a reviewer
2. a failing test.

Those were my two ideal cases. Thoughts?

[nicolai86]

Thanks for the clarification.

I'm not sure how much noise this would generate, as e.g. branches which are not yet ready for review will probably have lots of failing tests. Do you think it's a good idea to perform these actions before the author declares her branch ready to be reviewed?

[awitherow]

Nope, only if the review needed label is present.