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

Update CI after release of Java 13 #199

Merged
merged 3 commits into from
Dec 2, 2019
Merged

Update CI after release of Java 13 #199

merged 3 commits into from
Dec 2, 2019

Conversation

lorenzleutgeb
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Sep 20, 2019

Codecov Report

Merging #199 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #199   +/-   ##
=========================================
  Coverage     78.64%   78.64%           
  Complexity     1990     1990           
=========================================
  Files           141      141           
  Lines          6373     6373           
  Branches       1129     1129           
=========================================
  Hits           5012     5012           
  Misses          964      964           
  Partials        397      397

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a3fb4dc...60ba8f0. Read the comment docs.

@lorenzleutgeb
Copy link
Member Author

lorenzleutgeb commented Sep 20, 2019

OK, apparently I was a little too early. Requires Gradle 6.0 (according to gradle/gradle#8681), which has not been released yet. The 6.0 RC1 is due 2019-09-25. I guess we can just let this PR stick around until 6.0, and I mean not just an RC, is released and than update? :)

@@ -5,7 +5,7 @@
@rem you may not use this file except in compliance with the License.
@rem You may obtain a copy of the License at
@rem
@rem http://www.apache.org/licenses/LICENSE-2.0
Copy link
Member Author

Choose a reason for hiding this comment

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

Changes to this file were done automatically when upgrading the Gradle Wrapper.

@@ -7,7 +7,7 @@
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
Copy link
Member Author

Choose a reason for hiding this comment

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

Changes to this file were done automatically when upgrading the Gradle Wrapper.

@@ -1,5 +1,5 @@
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-5.3-all.zip
Copy link
Member Author

Choose a reason for hiding this comment

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

Changes to this file were done automatically when upgrading the Gradle Wrapper.

Copy link
Collaborator

@rtaupe rtaupe left a comment

Choose a reason for hiding this comment

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

Thank you, @lorenzleutgeb!

@rtaupe rtaupe merged commit 66267eb into master Dec 2, 2019
@rtaupe rtaupe deleted the java-13 branch December 2, 2019 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants