Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix OpenSSF Pinned-Dependencies Issues]: GitHub-owned GitHubAction actions/checkout not pinned by hash #10152

Merged
merged 1 commit into from
Feb 18, 2024

Conversation

liu-shaojun
Copy link
Contributor

@liu-shaojun liu-shaojun commented Feb 18, 2024

Description

1. Why the change?

This submission fixes OpenSSF Pinned-Dependencies Issues: GitHub-owned GitHubAction actions/checkout not pinned by hash (for example https://github.com/intel-analytics/BigDL/security/code-scanning/346)

https://github.com/actions/checkout/releases/tag/v2
https://github.com/actions/checkout/releases/tag/v3

@liu-shaojun liu-shaojun changed the title Fix: GitHub-owned GitHubAction not pinned by hash Fix OpenSSF Pinned-Dependencies Issues: GitHub-owned GitHubAction actions/checkout not pinned by hash Feb 18, 2024
@liu-shaojun liu-shaojun changed the title Fix OpenSSF Pinned-Dependencies Issues: GitHub-owned GitHubAction actions/checkout not pinned by hash [Fix OpenSSF Pinned-Dependencies Issues]: GitHub-owned GitHubAction actions/checkout not pinned by hash Feb 18, 2024
Copy link
Contributor

@glorysdj glorysdj left a comment

Choose a reason for hiding this comment

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

LGTM

@liu-shaojun liu-shaojun merged commit 291fdda into intel-analytics:main Feb 18, 2024
177 of 183 checks passed
@liu-shaojun liu-shaojun deleted the security branch February 18, 2024 08:49
Jasonzzt pushed a commit to Jasonzzt/BigDL that referenced this pull request Feb 19, 2024
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