Skip to content

ci: pin third-party GitHub Actions to verified release SHAs#644

Open
Vamshi-Microsoft wants to merge 1 commit into
mainfrom
ci/pin-third-party-actions
Open

ci: pin third-party GitHub Actions to verified release SHAs#644
Vamshi-Microsoft wants to merge 1 commit into
mainfrom
ci/pin-third-party-actions

Conversation

@Vamshi-Microsoft

Copy link
Copy Markdown
Contributor

Purpose

  • Pin third-party GitHub Actions to specific release commit SHAs (instead of mutable version tags) for reproducible and immutable workflow runs.
  • Replace the tj-actions/changed-files action in the broken-link checker with a built-in git diff step, which removes a third-party dependency from change detection.

Changes:

  • .github/workflows/broken-links-checker.yml — replace tj-actions/changed-files with a built-in git diff step; pin lycheeverse/lychee-action to 8646ba3 (v2.8.0).
  • .github/workflows/pr-title-checker.yml — pin amannn/action-semantic-pull-request to 48f2562 (v6.1.1).
  • .github/workflows/test.yml — pin MishaKav/pytest-coverage-comment to e48ae95 (v1.8.0).

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

@github-actions

Copy link
Copy Markdown

Coverage

Coverage Report •
FileStmtsMissCoverMissing
TOTAL121716186% 
report-only-changed-files is enabled. No files were changed during this commit :)

Tests Skipped Failures Errors Time
244 0 💤 0 ❌ 0 🔥 4.337s ⏱️

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 hardens CI workflows by pinning third-party GitHub Actions to immutable commit SHAs for reproducible runs, and removes a third-party dependency from markdown change detection in the broken links workflow.

Changes:

  • Replaced tj-actions/changed-files usage with a git diff-based change detection step for Markdown files.
  • Pinned lycheeverse/lychee-action, amannn/action-semantic-pull-request, and MishaKav/pytest-coverage-comment to specific release SHAs.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/broken-links-checker.yml Removes tj-actions/changed-files, adds git diff change detection, and pins lycheeverse/lychee-action to a SHA.
.github/workflows/pr-title-checker.yml Pins amannn/action-semantic-pull-request to a specific SHA.
.github/workflows/test.yml Pins MishaKav/pytest-coverage-comment to a specific SHA.

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

Comment thread .github/workflows/broken-links-checker.yml
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