Skip to content

Change all tags from workflows to pinned SHAs#152

Merged
jose-caballero merged 1 commit into
masterfrom
change-tag-to-sha
Apr 22, 2026
Merged

Change all tags from workflows to pinned SHAs#152
jose-caballero merged 1 commit into
masterfrom
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 (bcb83ba).
⚠️ Report is 3 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 8 times, most recently from 62fdbb4 to 9f27b31 Compare April 2, 2026 14:47
- Change all of the tags in the repo to pinned SHAs as it is best
  practice as a defence against supply chain attacks
@RyanH-STFC RyanH-STFC marked this pull request as ready for review April 22, 2026 09:13
@RyanH-STFC RyanH-STFC requested a review from a team as a code owner April 22, 2026 09:13
Comment thread .github/workflows/build_images.yaml
Comment thread .github/workflows/cloud_chatops.yaml
@jose-caballero jose-caballero merged commit aa9ef3b into master Apr 22, 2026
36 of 41 checks passed
@jose-caballero jose-caballero deleted the change-tag-to-sha branch April 22, 2026 09:40
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