Skip to content

DEVOPS-1133: Pin GitHub Actions to commit hashes - #456

Open
RomFloreani wants to merge 1 commit into
developfrom
DEVOPS-1133
Open

DEVOPS-1133: Pin GitHub Actions to commit hashes#456
RomFloreani wants to merge 1 commit into
developfrom
DEVOPS-1133

Conversation

@RomFloreani

@RomFloreani RomFloreani commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

DEVOPS-1133 - pin all GitHub actions and reusable workflows to hash
Expands moving GitHub Actions tags to the full semver tag pointing at the same commit, then pins every uses: to a commit hash with a dependabot-readable version comment.

Tags expanded in this repo:

  • MiraGeoscience/CI-tools/.github/workflows/reusable-jira-issue_to_jira.yml@v3 -> @v3.12.0
  • MiraGeoscience/CI-tools/.github/workflows/reusable-jira-pr_actions.yml@v3 -> @v3.12.0
  • MiraGeoscience/CI-tools/.github/workflows/reusable-python-static_analysis.yml@v3 -> @v3.12.0
  • MiraGeoscience/CI-tools/.github/workflows/reusable-python-pytest.yml@v3 -> @v3.12.0
  • MiraGeoscience/CI-tools/.github/workflows/reusable-python-publish_rattler_package.yml@v3 -> @v3.12.0
  • MiraGeoscience/CI-tools/.github/workflows/reusable-python-publish_pypi_package.yml@v3 -> @v3.12.0
  • MiraGeoscience/CI-tools/.github/workflows/reusable-python-release_conda_assets.yml@v3 -> @v3.12.0
  • MiraGeoscience/CI-tools/.github/workflows/reusable-python-release_pypi_assets.yml@v3 -> @v3.12.0
  • MiraGeoscience/CI-tools/.github/workflows/reusable-zizmor-advanced-security.yml@v3 -> @v3.12.0
  • MiraGeoscience/CI-tools/.github/workflows/reusable-zizmor-annotate.yml@v3 -> @v3.12.0

Workflows added:

  • .github/workflows/dependabot-auto-merge.yml
  • .github/workflows/wip-status.yml

All MiraGeoscience/CI-tools reusable workflows are pinned to 7241532854727d993872b67fc761139b8438615b (v3.12.0).

Copilot AI review requested due to automatic review settings July 29, 2026 16:41
@github-actions github-actions Bot changed the title Pin GitHub Actions to commit hashes DEVOPS-1133: Pin GitHub Actions to commit hashes Jul 29, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates this repository’s GitHub Actions reusable-workflow references to use the MiraGeoscience/CI-tools v3.9.1 tag instead of the floating v3 major tag across all workflows.

Changes:

  • Updated all uses: MiraGeoscience/CI-tools/...@v3 references to @v3.9.1.
  • Applied the same version bump consistently across security scan, Python analysis/deploy, and JIRA automation workflows.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
.github/workflows/security_scan.yml Updates Zizmor reusable workflow references from v3 to v3.9.1.
.github/workflows/python_deploy_prod.yml Updates Conda/PyPI production release reusable workflow references from v3 to v3.9.1.
.github/workflows/python_deploy_dev.yml Updates Conda/PyPI development publish reusable workflow references from v3 to v3.9.1.
.github/workflows/python_analysis.yml Updates static analysis and pytest reusable workflow references from v3 to v3.9.1.
.github/workflows/pr_jira_actions.yml Updates PR→JIRA reusable workflow reference from v3 to v3.9.1.
.github/workflows/issue_to_jira.yml Updates issue→JIRA reusable workflow reference from v3 to v3.9.1.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/security_scan.yml Outdated
Comment thread .github/workflows/python_deploy_prod.yml Outdated
Comment thread .github/workflows/python_deploy_dev.yml Outdated
Comment thread .github/workflows/python_analysis.yml Outdated
Comment thread .github/workflows/pr_jira_actions.yml Outdated
Comment thread .github/workflows/issue_to_jira.yml Outdated
@codecov

codecov Bot commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.45%. Comparing base (0053243) to head (4be23f2).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #456   +/-   ##
========================================
  Coverage    90.45%   90.45%           
========================================
  Files          113      113           
  Lines         6987     6987           
  Branches       862      862           
========================================
  Hits          6320     6320           
  Misses         451      451           
  Partials       216      216           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

name: Dependabot auto-merge

on:
pull_request_target: # zizmor: ignore[dangerous-triggers] no checkout, no execution of PR-authored code; only trusted event context is read

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pull_request_target: # zizmor: ignore[dangerous-triggers] no checkout, no execution of PR-authored code; only trusted event context is read
pull_request:

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

⚠️ Force push detected - Previous Copilot review comments may no longer be relevant to the current code. Please resolve any outdated threads and re-request a Copilot review when ready.

Pin all GitHub Actions and reusable workflow references to immutable
commit hashes, and add the dependabot auto-merge and WIP status workflows.
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

⚠️ Force push detected - Previous Copilot review comments may no longer be relevant to the current code. Please resolve any outdated threads and re-request a Copilot review when ready.

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