Skip to content

Fix zizmor findings in build.yml - #161

Merged
jwhui merged 1 commit into
openthread:mainfrom
jwhui:fix_zimor_output_actions
Jul 28, 2026
Merged

Fix zizmor findings in build.yml#161
jwhui merged 1 commit into
openthread:mainfrom
jwhui:fix_zimor_output_actions

Conversation

@jwhui

@jwhui jwhui commented Jul 28, 2026

Copy link
Copy Markdown
Member

Pin GitHub Actions to full commit SHAs in build.yml to resolve zizmor static analysis failures (unpinned-uses rule):

  • Pin jlumbroso/free-disk-space to SHA 54081f138730dfa15788a46383842cd2f914a1be (v1.3.1)
  • Pin actions/upload-artifact to SHA 043fb46d1a93c77aae656e7c1c64a875d1fc6a0a (v7.0.1)

Also add explicit top-level read-only permissions and set persist-credentials to false in actions/checkout step to satisfy zizmor security recommendations.

Pin GitHub Actions to full commit SHAs in build.yml to resolve
zizmor static analysis failures (unpinned-uses rule):

- Pin jlumbroso/free-disk-space to SHA
  54081f138730dfa15788a46383842cd2f914a1be (v1.3.1)
- Pin actions/upload-artifact to SHA
  043fb46d1a93c77aae656e7c1c64a875d1fc6a0a (v7.0.1)

Also add explicit top-level read-only permissions and set
persist-credentials to false in actions/checkout step to satisfy
zizmor security recommendations.
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@jwhui
jwhui merged commit c8b00eb into openthread:main Jul 28, 2026
8 checks passed
@jwhui
jwhui deleted the fix_zimor_output_actions branch July 28, 2026 15:44
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.

1 participant