豆豆友情提示:这是一个非官方 GitHub 代理镜像,主要用于网络测试或访问加速。请勿在此进行登录、注册或处理任何敏感信息。进行这些操作请务必访问官方网站 github.com。 Raw 内容也通过此代理提供。
Skip to content

Remove gradle_wrapper_updater feature flag#14174

Merged
kbukum1 merged 1 commit intomainfrom
cleanup_ff_gradle_wrapper_updater
Feb 16, 2026
Merged

Remove gradle_wrapper_updater feature flag#14174
kbukum1 merged 1 commit intomainfrom
cleanup_ff_gradle_wrapper_updater

Conversation

@kbukum1
Copy link
Copy Markdown
Contributor

@kbukum1 kbukum1 commented Feb 13, 2026

What are you trying to accomplish?

Remove the gradle_wrapper_updater feature flag to enable Gradle wrapper updates for all users by default.

This completes the rollout after successful validation in production with healthy metrics.

Fixes #2223

Anything you want to highlight for special attention from reviewers?

Test stubs were added for wrapper file requests in tests that didn't previously need them, since wrapper files are now always fetched.

How will you know you've accomplished your goal?

  • All existing tests pass
  • No remaining references to gradle_wrapper_updater in the codebase

Checklist

  • I have run the complete test suite to ensure all tests and linters pass.
  • I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
  • I have written clear and descriptive commit messages.
  • I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
  • I have ensured that the code is well-documented and easy to understand.

@kbukum1 kbukum1 requested a review from a team as a code owner February 13, 2026 19:45
@github-actions github-actions bot added the L: java:gradle Maven packages via Gradle label Feb 13, 2026
Comment thread .github/workflows/smoke.yml Outdated
@yeikel
Copy link
Copy Markdown
Contributor

yeikel commented Feb 13, 2026

@kbukum1 Can you review #14132 after this is merged?

@kbukum1 kbukum1 force-pushed the cleanup_ff_gradle_wrapper_updater branch from 4b2cf6d to 773b752 Compare February 13, 2026 23:09
@kbukum1
Copy link
Copy Markdown
Contributor Author

kbukum1 commented Feb 13, 2026

@kbukum1 Can you review #14132 after this is merged?

@yeikel , the PR is good. I left comment. It will be great to have green pipeline there. Can you do the temporary change?

#14132 (comment)

@kbukum1
Copy link
Copy Markdown
Contributor Author

kbukum1 commented Feb 14, 2026

Reverted temporary check after pipeline succeed. So it is ok to get error from Smoke / smoke (gradle-version-catalog) (pull_request) .

Uploading Screenshot 2026-02-14 at 12.39.39 PM.png…
Screenshot 2026-02-14 at 12 39 39 PM
orary

@kbukum1 kbukum1 force-pushed the cleanup_ff_gradle_wrapper_updater branch from ddeba92 to 0b31223 Compare February 14, 2026 18:44
@kbukum1 kbukum1 force-pushed the cleanup_ff_gradle_wrapper_updater branch from 0b31223 to 5b75f59 Compare February 16, 2026 22:02
@kbukum1 kbukum1 merged commit 5de5dcd into main Feb 16, 2026
101 of 103 checks passed
@kbukum1 kbukum1 deleted the cleanup_ff_gradle_wrapper_updater branch February 16, 2026 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

L: java:gradle Maven packages via Gradle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for updating gradle wrapper

3 participants