Skip to content

ci: remove flaky Gradle wrapper validation step#4

Merged
Manabu-GT merged 1 commit intomainfrom
fix/ci-gradle-wrapper-validation
Feb 13, 2026
Merged

ci: remove flaky Gradle wrapper validation step#4
Manabu-GT merged 1 commit intomainfrom
fix/ci-gradle-wrapper-validation

Conversation

@Manabu-GT
Copy link
Copy Markdown
Owner

@Manabu-GT Manabu-GT commented Feb 13, 2026

Summary

  • Remove gradle/wrapper-validation-action@v3 from both android-ci.yml and release.yml
  • This action frequently fails with ETIMEDOUT errors when GitHub Actions runners can't reach Gradle's checksum servers (known issue)
  • The Gradle wrapper is committed and trusted, so this validation adds no value

Test plan

  • Verify CI passes without the wrapper validation step

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Removed Gradle Wrapper validation step from CI/CD workflows to streamline the build pipeline.

The gradle/wrapper-validation-action frequently fails with ETIMEDOUT
errors when GitHub Actions runners can't reach Gradle's checksum servers.
Since the wrapper is committed and trusted, this check adds no value.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 13, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This pull request removes the Gradle Wrapper validation step (gradle/wrapper-validation-action@v3) from two CI/CD workflows, with control flow proceeding directly to JDK setup and subsequent build steps.

Changes

Cohort / File(s) Summary
Gradle Wrapper Validation Removal
.github/workflows/android-ci.yml, .github/workflows/release.yml
Removed the gradle/wrapper-validation-action@v3 step from both workflows; 3 lines deleted from each file.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/ci-gradle-wrapper-validation

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Manabu-GT Manabu-GT merged commit 814938c into main Feb 13, 2026
1 of 2 checks passed
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.

1 participant