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 version comments for pinned GitHub actions #1305

Merged
merged 5 commits into from
Dec 11, 2023

Conversation

spencerschrock
Copy link
Member

Fixing a minor annoyance from when I'm reviewing dependabot PRs. dependabot couldn't originally update the version comment, so they fell out of date. Today, dependabot can update version comments, but only does so if the old tag was correct, so I'm fixing the ones that are old.

There's more work to be done for .github/workflows/scorecards.yml, but it needs a overhaul of its own anyway.

Signed-off-by: Spencer Schrock <sschrock@google.com>
Signed-off-by: Spencer Schrock <sschrock@google.com>
Signed-off-by: Spencer Schrock <sschrock@google.com>
Signed-off-by: Spencer Schrock <sschrock@google.com>
@spencerschrock spencerschrock requested review from a team, raghavkaul and laurentsimon and removed request for a team December 1, 2023 19:42
Copy link

codecov bot commented Dec 1, 2023

Codecov Report

Merging #1305 (599e944) into main (16cadff) will not change coverage.
The diff coverage is n/a.

❗ Current head 599e944 differs from pull request most recent head 6ef4842. Consider uploading reports for the commit 6ef4842 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1305   +/-   ##
=======================================
  Coverage   63.17%   63.17%           
=======================================
  Files           4        4           
  Lines         296      296           
=======================================
  Hits          187      187           
  Misses         94       94           
  Partials       15       15           

@spencerschrock spencerschrock force-pushed the ci/fix-version-tags branch 2 times, most recently from 599e944 to 833e415 Compare December 4, 2023 21:54
@spencerschrock spencerschrock merged commit 75cb7af into ossf:main Dec 11, 2023
9 checks passed
@spencerschrock spencerschrock deleted the ci/fix-version-tags branch December 11, 2023 17:13
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.

2 participants