Pin wrapped actions by SHA - #10
Conversation
subosito/flutter-action@v2 and actions/cache@v6 are floating tags: anyone consuming Project516/setup-flutter@v1 runs whatever those tags point at the day their job starts, and this is published on the Marketplace. Pin both to full commit SHAs with the version in a trailing comment, the shape Renovate updates: subosito/flutter-action@1a449444c3... # v2.23.0 actions/cache@55cc8345863... # v6.1.0 Both SHAs verified against the tag refs via the GitHub API.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthrough
ChangesAction pinning
Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to This PR replaces floating wrapped-action versions with fixed commit SHAs, with no actionable merge-blocking risk remaining after normal checks and review. Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.) Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review |
1 similar comment
|
@coderabbitai review |
✅ Action performedReview finished.
|
Closes #8
action.ymlresolved its two dependencies by floating major tag(
subosito/flutter-action@v2,actions/cache@v6). This is published onthe Marketplace, so anyone using
Project516/setup-flutter@v1runswhatever those tags happen to point at on the day their job starts.
Pins both to full commit SHAs with the version in a trailing comment:
subosito/flutter-action@1a449444c387b1966244ae4d4f8c696479add0b2 # v2.23.0actions/cache@55cc8345863c7cc4c66a329aec7e433d2d1c52a9 # v6.1.0Both SHAs verified against
git/ref/tags/<tag>via the GitHub API beforecommitting.
renovate.jsonusesconfig:recommended, which alreadyunderstands this pin shape, so no config change is needed. The README
doesn't reference either wrapped action's version, so nothing to update
there.
Summary by CodeRabbit