Repository navigation
[LOW] Pin CI actions to immutable commits - #48
OskarEichler wants to merge 0 commit into
Conversation
|
First, why do you consider this an issue at all, considering how Tilt uses the actions? Explain what the worst possible outcome of this could be. Realize that this adds a maintenance burden by requiring a commit hash update every time I want to add a new Ruby version to CI. I realize that for many repositories, you want to specify the action commit hash, I'm just not sure the reasons for doing so apply to this repository. Second, I'm getting annoyed that you are ignoring my comments in rack/rack-test#360 (comment) regarding any PRs to any of my repositories. You have been warned previously in this repository: #32 (comment) . Note that including headings but not marking them as headings is not considered an appropriate way to "Do not use headings.". Consider this your final warning. I'll add an additionally requirement, don't include |
09203cd to
e6742eb
Compare
Summary
@v6and@v1.Impact
A moved or compromised mutable action ref can execute changed third-party code in Tilt's test jobs without a Tilt commit. The workflow is test-only, grants
contents: read, and declares no secrets or publishing credentials, so urgency is LOW.Verification
actions/checkout@v6resolves tod23441a48e516b6c34aea4fa41551a30e30af803.ruby/setup-ruby@v1resolves to95ef2b042f9d7a56d8268cba8559e2842e2ad01b.git diff --checkpasses.Breaking change
None. Runtime and package contents are unchanged.