Skip to content

Remove deprecated gradle/wrapper-validation-action#571

Closed
potiuk wants to merge 1 commit intomainfrom
fix/remove-deprecated-gradle-wrapper-validation
Closed

Remove deprecated gradle/wrapper-validation-action#571
potiuk wants to merge 1 commit intomainfrom
fix/remove-deprecated-gradle-wrapper-validation

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Mar 23, 2026

Summary

  • Remove gradle/wrapper-validation-action from dummy.yml and approved_patterns.yml
  • This old repo now redirects to gradle/actions/wrapper-validation, causing GitHub's enterprise allowlist to reject it as gradle/actions/wrapper-validation@v3.5.0 (not approved)
  • The newer gradle/actions/wrapper-validation@v5.0.2 is already present in the workflow and approved

Test plan

  • Verify the dummy workflow passes on this PR (no more allowlist rejection)
  • Confirm no other workflows reference gradle/wrapper-validation-action

🤖 Generated with Claude Code

Copy link
Copy Markdown
Member

@raboof raboof left a comment

Choose a reason for hiding this comment

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

please remove it from actions.yml, then that change should automatically propagate to the other two places.

The old gradle/wrapper-validation-action repo now redirects to
gradle/actions/wrapper-validation, causing GitHub's enterprise action
allowlist to reject it as gradle/actions/wrapper-validation@v3.5.0
(not in approved patterns). This broke the dummy workflow on all
branches. The newer gradle/actions/wrapper-validation@v5.0.2 is
already present and approved.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@potiuk potiuk force-pushed the fix/remove-deprecated-gradle-wrapper-validation branch from 0fdad9a to a8b0af7 Compare March 23, 2026 15:30
@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Mar 23, 2026

please remove it from actions.yml, then that change should automatically propagate to the other two places.

Oh yeah. I've been looking at it :)

@potiuk potiuk requested a review from raboof March 23, 2026 15:31
@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Mar 23, 2026

We have other older actions it seems.

@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Mar 23, 2026

The action astral-sh/setup-uv@eb1897b8dc4b5d5bfe39a428a8f2304605e0983c ... I guess this is transitive action ? Shall we enable it too ?

@raboof
Copy link
Copy Markdown
Member

raboof commented Mar 24, 2026

The action astral-sh/setup-uv@eb1897b8dc4b5d5bfe39a428a8f2304605e0983c ... I guess this is transitive action ? Shall we enable it too ?

I'm not sure what you mean - what does that have to do with this PR?

@raboof
Copy link
Copy Markdown
Member

raboof commented Mar 26, 2026

This was merged as #601 since

@raboof raboof closed this Mar 26, 2026
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.

2 participants