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

Remove stylecheck job #53162

Merged
merged 1 commit into from
Mar 18, 2025
Merged

Remove stylecheck job #53162

merged 1 commit into from
Mar 18, 2025

Conversation

r0mant
Copy link
Collaborator

@r0mant r0mant commented Mar 18, 2025

Remove stylecheck job because it uses reviewdog which was instrumental in the recent tj-actions/changed-files supply chain compromise: https://www.wiz.io/blog/new-github-action-supply-chain-attack-reviewdog-action-setup.

Verified

This commit was signed with the committer’s verified signature.
r0mant Roman Tkachenko
@r0mant
Copy link
Collaborator Author

r0mant commented Mar 18, 2025

@ptgott FYI we're removing stylecheck job to get rid of reviewdog as it was compromised.

@r0mant r0mant added this pull request to the merge queue Mar 18, 2025
Merged via the queue into master with commit 6c332af Mar 18, 2025
42 of 46 checks passed
@r0mant r0mant deleted the roman/nodog branch March 18, 2025 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants