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

fix: Upgrade Gradle and dependencies to fix Java 20 #573

Merged
merged 1 commit into from
Apr 28, 2023

Conversation

Myself5
Copy link
Contributor

@Myself5 Myself5 commented Apr 26, 2023

  • Add and configure refreshVersions module to update dependencies
  • Update dependencies to latest version for Gradle compatibility
  • Update Gradle to version 8.1.1

Note: Jacoco can not be automatically updated by the update and its version will need to be updated in the versions.properties manually.

Test: Build and run on ArchLinux using OpenJDK 20.0.1.

@andrii-bodnar
Copy link
Member

andrii-bodnar commented Apr 26, 2023

Hi @Myself5, thanks a lot for your contribution! it looks interesting

There are failed CI/CD checks in this PR, could you please take a look? Thanks

@andrii-bodnar
Copy link
Member

Also, I am wondering if it will be backward compatible with the previous CLI installations

@Myself5
Copy link
Contributor Author

Myself5 commented Apr 26, 2023

Hey @andrii-bodnar, working on the CI/CD issues.

I don't quite see how it would break any kind of previous CLI installations it only updates deps and the gradle tool similiar to what I did before, just that this time I added the refreshVersions module to make updating the dep-versions easier in the future.

@Myself5 Myself5 changed the title crowdin-cli: Upgrade Gradle to 8.1.1 fix: Upgrade Gradle and dependencies to fix Java 20 Apr 26, 2023
@Myself5 Myself5 force-pushed the gradle-8.1.1+refreshVersions branch from 2501958 to db73f88 Compare April 26, 2023 20:21
@codecov
Copy link

codecov bot commented Apr 26, 2023

Codecov Report

Merging #573 (49f8a73) into cli3 (0b9905f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               cli3     #573   +/-   ##
=========================================
  Coverage     61.52%   61.52%           
  Complexity     1150     1150           
=========================================
  Files           171      171           
  Lines          5044     5044           
  Branches        770      770           
=========================================
  Hits           3103     3103           
  Misses         1520     1520           
  Partials        421      421           

build.gradle Outdated Show resolved Hide resolved
* Add and configure [refreshVersions](https://jmfayard.github.io/refreshVersions/) module to update dependencies
* Update dependencies to latest version for Gradle compatibility
* Update Gradle to version 8.1.1

Note: Jacoco can not be automatically updated by the update and its version will need to be updated in the versions.properties manually.

Test: Build and run on ArchLinux using OpenJDK 20.0.1.
@Myself5 Myself5 force-pushed the gradle-8.1.1+refreshVersions branch from db73f88 to 49f8a73 Compare April 27, 2023 14:47
Copy link
Member

@andrii-bodnar andrii-bodnar left a comment

Choose a reason for hiding this comment

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

@Myself5 thank you!

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