-
Notifications
You must be signed in to change notification settings - Fork 280
🌱 Don't run golanci-lint update on PRs #2659
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
Conversation
✅ Deploy Preview for kubernetes-sigs-cluster-api-openstack ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
/hold |
e20ed2b
to
ca48b37
Compare
Ok now it is at least trying to create the PR! The remaining issue is that the default token doesn't have permission to create PRs when the action is triggered from a PR/push. I think that is a sane policy that we should not tamper with. Perhaps we can do some kind of dry-run on PRs 🤔 |
ca48b37
to
6f08cea
Compare
We cannot run this on PRs since that would require more privileges. We can still manually trigger it to run on a specific branch as a way of testing the workflow. Also fix the syntax for add-paths. It should be a comma or newline separated list of paths, but we had an extra "-" in front of the path. Signed-off-by: Lennart Jern <[email protected]>
6f08cea
to
b10b418
Compare
I have not found any clever way to dry-run this so removing the PR trigger. We can still trigger the workflow manually on specific branches as a way of testing it. |
/lgtm Thanks! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: EmilienM The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
What this PR does / why we need it:
The workflow is failing in #2647 because the base is not set. This is an attempt to fix that.
However, creating a PR from a workflow triggered by a PR requires more privileges than the normal token has. I suggest we don't run this workflow on PRs at all, rather than giving it more permissions and a custom token.
We can still manually trigger it to run and select on which branch it should run.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
TODOs: