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

Upgrade to latest static analysis Action #7

Merged
merged 1 commit into from
May 7, 2024

Conversation

nwiltsie
Copy link
Member

@nwiltsie nwiltsie commented May 6, 2024

This PR replaces the existing CI/CD workflow in this repository with the Action-ified version from tool-static-analysis. Under the hood, the Action is using the exact same docker image to perform the checks, so this upgrade should have no visible change.

Turning the static analysis checks into an Action makes it much easier for your friendly neighborhood Infrastructure Team to make large-scale updates without opening pull requests in dozens or hundreds of repositories.

As an example, 247 workflows across the uclahs-cds organization are using actions/checkout@v2 or actions/checkout@v3. Those will stop working on May 13, 2024. This PR (and its dozens of identical siblings) centralize usage of tool-static-analysis@v1, which internally uses actions/checkout@v4. In the future, we will only need to update tool-static-analysis in order for all the repositories updated in this wave to reap the benefits.

@nwiltsie nwiltsie added the standardization Standardization task label May 6, 2024
@nwiltsie nwiltsie merged commit a0e80b1 into main May 7, 2024
5 checks passed
@nwiltsie nwiltsie deleted the nwiltsie-static-upgrade branch May 7, 2024 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
standardization Standardization task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant