Skip to content

feat(superfluous-actions): allow rust-toolchain on self-hosted runners - #2405

Open
ubiratansoares wants to merge 1 commit into
zizmorcore:mainfrom
ubiratansoares:u/rust-toolchain-self-hosted
Open

ubiratansoares wants to merge 1 commit into
zizmorcore:mainfrom
ubiratansoares:u/rust-toolchain-self-hosted

Conversation

@ubiratansoares

Copy link
Copy Markdown
Contributor

Pre-submission checks

Please check these boxes:

  • Mandatory: This PR corresponds to an issue (if not, please create
    one first).

  • Having read the AI policy, I hereby disclose the use of an LLM or other
    AI coding assistant in the creation of this PR. PRs will not be rejected
    for using AI tools, but will be rejected for undisclosed use or
    use that violates the policy.

If a checkbox is not applicable, you can leave it unchecked.

AI disclosure

I did not use AI to write this PR

Summary

Easy follow-up of #2350.

Closes #1865

Test Plan

Added a snapshot test covering this case :)

Comment on lines +67 to +70
enum SkipCriteria {
Never,
SelfHostedRunner,
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@woodruffw I don't necessarily love adding a new column to SUPERFLUOUS_ACTIONS, but at least an enum can easily embrance other skip criteria we may get in the future.

Let me know your thoughts on this :)

This branch has not been deployed

No deployments
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.

[BUG]: False positive superfluous-actions for dtolnay/rust-toolchain on self-hosted runner

1 participant