Skip to content
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

Update the scorecard dependencies a little higher (again) #846

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

aiuto
Copy link
Collaborator

@aiuto aiuto commented Apr 2, 2024

Yun. What do you think about a recommendation that all bazelbuild repositories copy this workflow directly from bazel?

@aiuto aiuto requested a review from meteorcloudy as a code owner April 2, 2024 17:46
@meteorcloudy
Copy link
Member

Yes, I think this makes sense, and should probably be added to https://docs.google.com/document/d/1L1JFgjpZ7SrBinb24DC_5nTIELeYDacikcme-YcA7xs/edit#heading=h.5kq5q3o81vg9

@aiuto
Copy link
Collaborator Author

aiuto commented Apr 3, 2024

Yes, I think this makes sense, and should probably be added to https://docs.google.com/document/d/1L1JFgjpZ7SrBinb24DC_5nTIELeYDacikcme-YcA7xs/edit#heading=h.5kq5q3o81vg9

Suggestion added in the doc

@aiuto aiuto changed the title roll them forward again Update the scorecard dependencies a little higher (again) Apr 4, 2024
@aiuto
Copy link
Collaborator Author

aiuto commented Apr 4, 2024

I copied all the dependencies from bazel. I can't use that scorecard.yml directly because of diffs like the branch name and run frequency.

@aiuto aiuto requested a review from cgrindel April 4, 2024 14:45
@aiuto
Copy link
Collaborator Author

aiuto commented Apr 16, 2024

Github is wedge. The conversation is resolved, but the merge is still blocked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants