-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Fix #24
Fix #24
Conversation
@@ -8,8 +8,8 @@ inputs: | |||
github-token: | |||
description: 'GitHub Token' | |||
required: true | |||
number: | |||
description: 'Pull Request number' |
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.
please don't remove existing features
@@ -0,0 +1,16 @@ | |||
## Proposed Changes |
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.
please remove this, it has nothing to do with your change
} | ||
|
||
main().catch(err => core.setFailed(err.message)) | ||
main().catch(err => core.setFailed(err.message)); |
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.
please don't change the code style
if (!labels.includes(approveLabel)) { | ||
console.log("PR does not include approve-label, exiting"); | ||
return; | ||
} |
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.
I think you're changing this action to something else, it currently is used for approving arbitrary PRs by number, and it looks like you're proposing to change it to one that will auto-approve PRs if they contain a certain label.
Last time I checked there are already actions for this on the marketplace.
automatically merge.