Skip to content

Conversation

@JonasPollokZweitag
Copy link
Contributor

No description provided.

Copy link
Member

@kevinkupski kevinkupski left a comment

Choose a reason for hiding this comment

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

Thanks for opening the PR. I did not review the template/Python scripts in detail but would test the workflow(s) when they are ready to check the result.

Btw. do you think we can run this whenever this code changes to have a test for the workflow(s)? Similar to the other workflow/action tests?

@kevinkupski
Copy link
Member

It also looks like that there are some syntax errors: https://github.com/zweitag/github-actions/actions/runs/20659089108

@JonasPollokZweitag JonasPollokZweitag self-assigned this Jan 14, 2026
@JonasPollokZweitag
Copy link
Contributor Author

JonasPollokZweitag commented Jan 20, 2026

I made the suggested changes. You can check the Trivy Scan Report here and the Checkov scan here.

Copy link
Member

@kevinkupski kevinkupski left a comment

Choose a reason for hiding this comment

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

Very good work 👏
I have just some small last comments. After fixing it, feel free to merge this PR.

@JonasPollokZweitag JonasPollokZweitag merged commit 6f9a6a3 into main Jan 27, 2026
11 checks passed
@JonasPollokZweitag JonasPollokZweitag deleted the chore--create-configurable-Security-Scan branch January 27, 2026 08:01
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