Skip to content
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

Prepare several Gradle 9.0 deprecations #5686

Merged
merged 6 commits into from
Jun 10, 2024
Merged

Conversation

sh-cho
Copy link
Contributor

@sh-cho sh-cho commented May 18, 2024

Motivation:

Prepare deprecations which will be removed or fail in Gradle 9.0

Modifications:

Result:

  • Several deprecations of Gradle 9.0 are handled in advance

@minwoox
Copy link
Member

minwoox commented May 20, 2024

@sh-cho Thanks!
Could you also send a pr to line/gradle-scripts#177?

@sh-cho
Copy link
Contributor Author

sh-cho commented May 20, 2024

@sh-cho Thanks! Could you also send a pr to line/gradle-scripts#177?

Oh I just replaced all 😅
I will make a PR in gradle scripts repo for that

`JavaExecSpec.main` property is deprecated. Use the `mainClass` property
instead.
The `AbstractArchiveTask.archivePath` property has been deprecated. This
is scheduled to be removed in Gradle 9.0.

https://docs.gradle.org/8.7/dsl/org.gradle.api.tasks.bundling.AbstractArchiveTask.html
It will be handled in the `gradle-scripts` repository.
Copy link
Contributor

@jrhee17 jrhee17 left a comment

Choose a reason for hiding this comment

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

👍 👍 👍

subrepo:
  subdir:   "gradle/scripts"
  merged:   "f716cfe16"
upstream:
  origin:   "https://github.com/line/gradle-scripts"
  branch:   "main"
  commit:   "f716cfe16"
git-subrepo:
  version:  "0.4.5"
  origin:   "https://github.com/ingydotnet/git-subrepo"
  commit:   "aa416e4"
Copy link
Member

@minwoox minwoox left a comment

Choose a reason for hiding this comment

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

Thanks, @sh-cho!
I pushed a commit that fetches the upstream change.
line/gradle-scripts@f716cfe

@minwoox minwoox added this to the 1.29.0 milestone Jun 4, 2024
subrepo:
  subdir:   "gradle/scripts"
  merged:   "a3211a7ec"
upstream:
  origin:   "https://github.com/line/gradle-scripts"
  branch:   "main"
  commit:   "a3211a7ec"
git-subrepo:
  version:  "0.4.6"
  origin:   "https://github.com/Homebrew/brew"
  commit:   "e130e47f23"
Copy link
Member

@trustin trustin left a comment

Choose a reason for hiding this comment

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

Thanks, @sh-cho! 🙇

@minwoox minwoox merged commit b3da82e into line:main Jun 10, 2024
14 of 15 checks passed
@minwoox
Copy link
Member

minwoox commented Jun 10, 2024

Thanks a lot, @sh-cho! 👍

@sh-cho sh-cho deleted the prepare-gradle-9.0 branch June 10, 2024 11:48
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.

None yet

4 participants