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

[test] test zizmor #103

Closed
wants to merge 1 commit into from
Closed

[test] test zizmor #103

wants to merge 1 commit into from

Conversation

lazka
Copy link
Member

@lazka lazka commented Dec 13, 2024

No description provided.

@lazka
Copy link
Member Author

lazka commented Dec 13, 2024

hm, that doesn't do anything

@lazka
Copy link
Member Author

lazka commented Dec 13, 2024

I've enabled it but it doesn't do anything at all, there is no indication in this PR that there is a warning written by zizmor:

image

I guess I'll remove sarif again

@lazka lazka closed this Dec 13, 2024
@lazka lazka reopened this Dec 13, 2024
@lazka lazka marked this pull request as draft December 13, 2024 20:25
@lazka
Copy link
Member Author

lazka commented Dec 13, 2024

I've changed things to just fail on exit code in c5b593a since this is not working.

@lazka lazka closed this Dec 13, 2024
@lazka lazka reopened this Dec 13, 2024
@lazka lazka marked this pull request as ready for review December 13, 2024 20:56
@lazka lazka closed this Dec 13, 2024
@lazka lazka reopened this Dec 13, 2024
@lazka lazka closed this Dec 13, 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.

1 participant