Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Address zizmor security issues in GitHub Actions setup #121

Merged
merged 2 commits into from
Feb 2, 2025
Merged

Address zizmor security issues in GitHub Actions setup #121

merged 2 commits into from
Feb 2, 2025

Conversation

lopopolo
Copy link
Member

@lopopolo lopopolo commented Feb 2, 2025

There's a slight breaking change in here where the check-minimal-versions action no longer installs clippy and rustup, but these should be unused by callers.

@lopopolo lopopolo added A-github-action Area: GitHub Action API and configuration. A-rustup Area: Configuring Rust toolchains with Rustup. A-security Area: Security vulnerabilities and unsoundness issues. labels Feb 2, 2025
@lopopolo lopopolo merged commit a6c04c3 into trunk Feb 2, 2025
12 checks passed
@lopopolo lopopolo deleted the dev/lopopolo-zizmor-green branch February 2, 2025 01:09
lopopolo added a commit that referenced this pull request Feb 2, 2025
@lopopolo
Install llvm-tools-preview in code coverage action
ec58d0b 
copy and paste error when fixing up zizmor issues in #121.
@lopopolo lopopolo mentioned this pull request Feb 2, 2025
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
A-github-action Area: GitHub Action API and configuration. A-rustup Area: Configuring Rust toolchains with Rustup. A-security Area: Security vulnerabilities and unsoundness issues.
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lopopolo