Skip to content

Change all tags from workflows to pinned SHAs#152

Draft
RyanH-STFC wants to merge 1 commit intomasterfrom
change-tag-to-sha
Draft

Change all tags from workflows to pinned SHAs#152
RyanH-STFC wants to merge 1 commit intomasterfrom
change-tag-to-sha

Conversation

@RyanH-STFC
Copy link
Copy Markdown
Contributor

  • Change all of the tags in the repo to pinned SHAs as it is best practice as a defence against supply chain attacks

Description:


Submitter:

Have you done the following?:

  • Labeled the pull request from the following? major | minor | patch or documentation | workflow
  • Updated the documentation?
  • Added unit tests for new / untested code?
  • Deployed the changes onto dev-chatops.nubes.rl.ac.uk?

New / Existing features:

  • Written integration tests?
  • Configured the Slack Application with new commands or permission scope changes?
  • Updated the version.txt and docker-compose.yml files?

Reviewer:

Have you checked the following?:

  • Version change is appropriate to the code changes? Semantic versioning documentation here
  • Unit test code coverage is acceptable?
  • Do the CI jobs pass?

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.24%. Comparing base (cae7371) to head (8373ba9).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #152      +/-   ##
==========================================
- Coverage   99.79%   97.24%   -2.56%     
==========================================
  Files          20       16       -4     
  Lines         984     1198     +214     
==========================================
+ Hits          982     1165     +183     
- Misses          2       33      +31     
Flag Coverage Δ
rabbit_consumer 97.24% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@RyanH-STFC RyanH-STFC force-pushed the change-tag-to-sha branch 7 times, most recently from e8383f9 to 62fdbb4 Compare April 2, 2026 14:38
- Change all of the tags in the repo to pinned SHAs as it is best
  practice as a defence against supply chain attacks
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.

1 participant