-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Remove gradle-wrapper.jar from source tree
#36771
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Summary of ChangesHello @snazy, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request modernizes the project's Gradle setup by removing the Highlights
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
As discussed with @snazy, I will review it. |
|
Assigning reviewers: R: @chamikaramj for label java. Note: If you would like to opt out of this review, comment Available commands:
The PR bot will only process comments in the main thread (not review comments). |
|
Reminder, please take a look at this pr: @chamikaramj @liferoad |
|
Assigning new set of reviewers because Pr has gone too long without review. If you would like to opt out of this review, comment R: @ahmedabu98 for label java. Available commands:
|
|
Reminder, please take a look at this pr: @ahmedabu98 @Abacn |
|
Assigning new set of reviewers because Pr has gone too long without review. If you would like to opt out of this review, comment R: @kennknowles for label java. Available commands:
|
|
stop reviewer notifications |
|
Stopping reviewer notifications for this pull request: requested by reviewer. If you'd like to restart, comment |
|
This seems like a mostly-benign change. It is also fine to simply exclude the jar from the source release zip (I'm actually surprised if this is not already the case). The github action upgrades are nice and I don't think they require discussion - could they be decoupled and committed first? |
1399f99 to
6e88956
Compare
This change replaces the now unmaintained `gradle/gradle-build-action` with `gradle/actions/setup-gradle`. Also remove the unmaintained `gradle/wrapper-validation-action` w/o a replacement, assuming apache#36771 is going in.
6e88956 to
846b8fe
Compare
|
@kennknowles sure! I've just rebased this branch and split it into 2 commits, the 1st commit is the content of #37064. |
This change replaces the now unmaintained `gradle/gradle-build-action` with `gradle/actions/setup-gradle`. Also remove the unmaintained `gradle/wrapper-validation-action` w/o a replacement, assuming #36771 is going in.
The ASF Release Policy mentions that a source release should not contain compiled code. Jar files, like the `gradle/wrapper/gradle-wrapper.jar` file, are compiled code and should not be included in source releases. The Gradle build tool currently offers no way out-of-the box to run Gradle without having the wrapper jar in the source tree. [The repository](https://github.com/snazy/gradle-wrapper-no-jar) demonstrates how to use Gradle without having the wrapper jar in the source tree. This change adopts these changes for Apache Beam, and also * Replace the deprecated [`gradle/gradle-build-action`](https://github.com/gradle/gradle-build-action) (repo is archived/read-only) with [`gradle/actions/setup-gradle`](https://github.com/gradle/actions/tree/main/setup-gradle) * Remove usage of the no longer necessary [`gradle/wrapper-validation-action`](https://github.com/gradle/wrapper-validation-action) (repo is archived/read-only)
846b8fe to
72f9e74
Compare
|
Rebased (w/o changes) since #37064 has been merged. |
The ASF Release Policy mentions that a source release should not contain compiled code. Jar files, like the
gradle/wrapper/gradle-wrapper.jarfile, are compiled code and should not be included in source releases. The Gradle build tool currently offers no way out-of-the box to run Gradle without having the wrapper jar in the source tree.The repository demonstrates how to use Gradle without having the wrapper jar in the source tree.
This change adopts these changes for Apache Beam, and also
gradle/gradle-build-action(repo is archived/read-only) withgradle/actions/setup-gradlegradle/wrapper-validation-action(repo is archived/read-only)