Skip to content

ci: Fix zizmor warnings and bump actions - #447

Merged
dervoeti merged 8 commits into
stackabletech:mainfrom
Saul-STFC:CI-Hardening
Sep 2, 2026
Merged

ci: Fix zizmor warnings and bump actions#447
dervoeti merged 8 commits into
stackabletech:mainfrom
Saul-STFC:CI-Hardening

Conversation

@Saul-STFC

Copy link
Copy Markdown
Contributor

This PR explicitly defines permissions fixing zizmor warnings and also bumps deps

@Techassi Techassi moved this to Development: In Review in Stackable Engineering Aug 31, 2026
@Techassi Techassi changed the title CI: Fix zizmor warnings and bump deps ci: Fix zizmor warnings and bump actions Aug 31, 2026
Techassi
Techassi previously approved these changes Aug 31, 2026
Comment thread .github/workflows/reusable_build_image.yaml Outdated
Comment thread .github/workflows/reusable_build_image.yaml Outdated
Techassi
Techassi previously approved these changes Sep 1, 2026

@Techassi Techassi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved.

CC'ing @dervoeti for a second pair of eyes.

@dervoeti dervoeti left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Renamed the input params, rest LGTM (not sure if my suggestions work since they both affect lines that were not modified in this PR)

Comment thread .github/workflows/reusable_build_image.yaml Outdated
Comment thread .github/workflows/reusable_build_image.yaml Outdated
Co-authored-by: Lukas Krug <lukas@luvosys.de>
@Techassi
Techassi requested a review from dervoeti September 2, 2026 14:56
@dervoeti
dervoeti merged commit b451926 into stackabletech:main Sep 2, 2026
2 checks passed
@Techassi Techassi moved this from Development: In Review to Development: Done in Stackable Engineering Sep 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Done

Development

Successfully merging this pull request may close these issues.

3 participants