-
-
Notifications
You must be signed in to change notification settings - Fork 123
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
feat: add ignoreLabels option to force ignore errors #174
Conversation
@amannn Hello, could you please take a look at this? 🙏 |
Signed-off-by: Kenji Miyake <[email protected]>
80e0ac2
to
6c82aeb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for another contribution @kenji-miyake! The implementation looks pretty good, I only left some minor comments.
Co-authored-by: Jan Amann <[email protected]>
Signed-off-by: Kenji Miyake <[email protected]>
Signed-off-by: Kenji Miyake <[email protected]>
Signed-off-by: Kenji Miyake <[email protected]>
Signed-off-by: Kenji Miyake <[email protected]>
Signed-off-by: Kenji Miyake <[email protected]>
Signed-off-by: Kenji Miyake <[email protected]>
Signed-off-by: Kenji Miyake <[email protected]>
Co-authored-by: Jan Amann <[email protected]>
Co-authored-by: Jan Amann <[email protected]>
2cad533
to
67c0d96
Compare
Signed-off-by: Kenji Miyake <[email protected]>
67c0d96
to
2ad53de
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really good! Thank you for another great contribution! 👏👏
🎉 This PR is included in version 4.3.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
It is useful to have a feature that can force ignore errors when a certain label is added.
It can be used to support the cases shown in #159 with the
validateSingleCommitMatchesPrTitle
option enabled.I've tested this in kenji-miyake#7.