Skip to content

[BUILD]: Add commit hashes to engine-benchmarks workflow to pin actions#830

Merged
MacOS merged 6 commits into
SKaiNET-developers:developfrom
MacOS:engine-benchmarks-workflow/add-commit-hashes
Jul 16, 2026
Merged

[BUILD]: Add commit hashes to engine-benchmarks workflow to pin actions#830
MacOS merged 6 commits into
SKaiNET-developers:developfrom
MacOS:engine-benchmarks-workflow/add-commit-hashes

Conversation

@MacOS

@MacOS MacOS commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

This PR pins all actions in the engine-benchmarks.yml workflow with their commit hash to increase security (#815 ).

The OpenSSF Score should increase after this PR has been merged.

@MacOS
MacOS requested review from Copilot and michalharakal July 16, 2026 10:40
@MacOS MacOS self-assigned this Jul 16, 2026
@MacOS MacOS added enhancement New feature or request github_actions Pull requests that update GitHub Actions code labels Jul 16, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Pins GitHub Actions used by the engine-benchmarks workflow to immutable commit SHAs to improve supply-chain security and help raise the repository’s OpenSSF Score.

Changes:

  • Replace actions/checkout, actions/setup-java, and actions/upload-artifact version tags with full commit hashes in engine-benchmarks.yml.
  • Add inline comments retaining the original action versions for readability/auditing.

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

Comment thread .github/workflows/engine-benchmarks.yml Outdated

- name: Set up JDK 21
uses: actions/setup-java@v5
uses: actions/setup-java@dded0888837ed1f317902acf8a20df0ad188d165 # v5.0.0

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Done.

… more consistent with the other inline comments

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@michalharakal michalharakal 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.

Thank you @MacOS

@MacOS
MacOS merged commit c08e40f into SKaiNET-developers:develop Jul 16, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants