Skip to content

Feature/79895 Split build and release in two separate Github Actions jobs#26

Closed
bogdandina wants to merge 11 commits into
mainfrom
feature/79895_split-build-and-release-in-separate-jobs
Closed

Feature/79895 Split build and release in two separate Github Actions jobs#26
bogdandina wants to merge 11 commits into
mainfrom
feature/79895_split-build-and-release-in-separate-jobs

Conversation

@bogdandina
Copy link
Copy Markdown
Contributor

No description provided.

bogdandina and others added 7 commits May 19, 2026 22:43
Add a zizmor job to ci.yml that runs on every PR and push to main,
auditing all workflow files for unpinned actions, template injection,
excessive permissions, and other insecure patterns via
zizmorcore/zizmor-action (SARIF results uploaded to GitHub Security tab).

Pin every uses: reference across all four shared workflows to an
immutable commit SHA with a human-readable version comment, eliminating
the supply-chain risk of mutable tags being silently redirected to
malicious commits.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@bogdandina bogdandina requested a review from haphut May 19, 2026 20:23
Comment thread .github/workflows/ci-cd-java.yml Fixed
Comment thread .github/workflows/ci-cd-kotlin.yml Fixed
Comment thread .github/workflows/ci-cd-typescript.yml Fixed
Comment thread .github/workflows/ci-cd-java.yml
Comment thread .github/workflows/ci-cd-typescript.yml
Copy link
Copy Markdown
Contributor

@haphut haphut left a comment

Choose a reason for hiding this comment

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

Great job! I'm approving even though I'd like to see some changes. If you agree with the changes, you can merge after them. Otherwise let's discuss.

@haphut
Copy link
Copy Markdown
Contributor

haphut commented May 21, 2026

I think you can close the Zizmor PR as the commits are already here, as you mentioned, and they look fine as well.

@bogdandina bogdandina closed this May 27, 2026
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.

3 participants