You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Yes, I understand that the GitHub action repository is not the repository of golangci-lint itself.
Yes, I've searched similar issues on GitHub and didn't find any.
Your feature request related to a problem? Please describe.
Currently the only-new-issues only supports a few event types like pull_request, pull_request_target, push, merge_group.
The pull_request_review event is sometimes used to trigger CI only when the pull request is approved (using with types: submitted).
This comes with a few benefits:
prevent any malicious code from triggering your GItHub Actions
cuts down the total billable minutes (since you're only running lints on pull request approvals and not on every update to the PR)
Describe the solution you'd like.
support the pull_request_review type (could be confined to a few types like submitted, edited etc)
cuts down the total billable minutes (since you're only running lints on pull request approvals and not on every update to the PR)
This only applies to private repositories so your first argument doesn't fit.
Anyway, pull_request_review is not for approval but for reviews.
Even if you restrict only on approval (i.e. submitted + if: github.event.review.state == 'APPROVED'), this means you will approve a PR without having the result of the linting, this doesn't make sense.
If you think of another activity type (ex: edited), it's the same thing but worse.
To cut down the total billable minutes:
run golangci-lint locally before a push (and before a commit), the cache will do the job.
don't push half-done work
You want to launch golangci-lint on demand but your proposal (pull_request_review) is a bad approach.
Welcome
Your feature request related to a problem? Please describe.
Currently the
only-new-issues
only supports a few event types likepull_request
,pull_request_target
,push
,merge_group
.The
pull_request_review
event is sometimes used to trigger CI only when the pull request is approved (using with types: submitted).This comes with a few benefits:
Describe the solution you'd like.
support the
pull_request_review
type (could be confined to a few types like submitted, edited etc)Additional context.
ref: #1059
The text was updated successfully, but these errors were encountered: