-
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
Workflows failing due repo config #908
Comments
Sending a PR should be the best choice |
Also the update core index.js is failing: I assume the same will happen with Update Npm Index as it will require write permissions. |
Maybe the best option for now is disable the protection to a level that the workflows can commit to main? |
As agreed on last meeting, the approach will be to migrate the Github actions with writing permissions to send a Pr with the changes:
In the meantime I can keep my fork running the actions and I can PR the changes until we make the refactor as my fork does not includes the branch protections. This won't impact negatively in the scorecard as it is a Pr that we will review 😉 |
Related to: - nodejs#908 - nodejs#949
Related to: - nodejs#908 - nodejs#949
* feat: Added PR generation capabilities to core index pipeline Related: #908 * chore: changed dependency
* feat: Added PR generation capabilities to npm index pipeline Related: #908 * chore: changed dependency
I close the issues as the workflows are working fine now 🎉 |
…e (#997) * feat: Added PR generation capabilities to reporting scorecard pipelinere Related: nodejs/security-wg#908 * chore: changed dependency
…e (#997) * feat: Added PR generation capabilities to reporting scorecard pipelinere Related: nodejs/security-wg#908 * chore: changed dependency
The OpenSSF Scoring workflow is failing has we had changed the rules in the repo to make
main
a protected branch:You can check the settings here. I believe that we might make it work by disable the
Do not allow bypassing the above settings
option. But this might impact the scorecard for nodejs/security-wgThe text was updated successfully, but these errors were encountered: