-
Notifications
You must be signed in to change notification settings - Fork 517
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
Add OpenSSF Scorecard #789
Conversation
# - you want to enable the Branch-Protection check on a *public* repository, or | ||
# - you are installing Scorecard on a *private* repository | ||
# To create the PAT, follow the steps in https://github.com/ossf/scorecard-action?tab=readme-ov-file#authentication-with-fine-grained-pat-optional. | ||
repo_token: ${{ secrets.SCORECARD_TOKEN }} |
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 dont see this secret in the repo?
Also we have branch protection enabled, so how will that work?
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.
Good question. It ignores this secret if it does not exist, I added it in case we wanted to go down that route, but doesn't matter that it isn't there.
I added a comment on the work item about this too: https://dev.azure.com/CSUSolEng/Azure%20Landing%20Zones/_workitems/edit/35169#14168293
In summary we need to move away from legacy branch protection rules to using rulesets instead and then we can forget about this secret.
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.
As discussed on the call with @jaredfholgate, we are good at this point.
* Create scorecard.yml * Update README.md
Overview/Summary
This PR adds the OpenSSF scorecard and associated badge.
This PR fixes/adds/changes/removes
N/A
Breaking Changes
N/A
Testing Evidence
N/A
As part of this Pull Request I have
.bicep
file/s I am adding/editing are using the latest API version possiblemain
branch