-
Notifications
You must be signed in to change notification settings - Fork 114
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(land): add skipChecks option #475
base: main
Are you sure you want to change the base?
Conversation
skipChecks will allow skipping certain checks. Useful to skip "commit after last approval" on commit queue. Ref: nodejs/node#34770
I'm not 100% sure we're going to use this, but I guess it doesn't hurt to have it either. |
Codecov Report
@@ Coverage Diff @@
## master #475 +/- ##
==========================================
+ Coverage 82.56% 82.70% +0.13%
==========================================
Files 34 34
Lines 1658 1671 +13
==========================================
+ Hits 1369 1382 +13
Misses 289 289
Continue to review full report at Codecov.
|
@@ -42,6 +45,21 @@ const landOptions = { | |||
describe: 'Land a backport PR onto a staging branch', | |||
default: false, | |||
type: 'boolean' | |||
}, | |||
skipChecks: { |
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.
We should add this option to documentation as well, perhaps with an accompanying example :)
@mmarchini could you rebase this? |
This PR is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made. |
skipChecks will allow skipping certain checks. Useful to skip "commit
after last approval" on commit queue.
Ref: nodejs/node#34770