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

Document minimal Gradle version. #478

Merged
merged 1 commit into from
Aug 14, 2018
Merged

Conversation

OndraZizka
Copy link
Contributor

@OndraZizka OndraZizka commented Jul 22, 2018

4.9 is the current. Maybe the whole 4.x works but 3.x does not.
This is an issue mainly because Linux distributions (e.g. Ubuntu) still have 3.x in the packages.

4.9 is the current. Maybe the whole 4.x works  but 3.x does not.
@coveralls
Copy link

coveralls commented Jul 22, 2018

Coverage Status

Coverage remained the same at 80.08% when pulling 7dc6c40 on OndraZizka:patch-1 into a402b14 on jbake-org:master.

@ancho
Copy link
Member

ancho commented Jul 23, 2018

Yes. I noticed this while working on #471. The pr includes an update to gradle 4.9.

@jonbullock
Copy link
Member

Thanks for this 👍

@jonbullock jonbullock added this to the v2.6.2 milestone Jul 31, 2018
@ancho
Copy link
Member

ancho commented Aug 1, 2018

Another thing. If one uses the gradle wrapper (gradlew) there should be no problem with gradle version conflicts. With exception of >=4.0 and <= 4.9 and java 7.

@jonbullock jonbullock self-assigned this Aug 14, 2018
@jonbullock jonbullock merged commit 5748e07 into jbake-org:master Aug 14, 2018
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.

4 participants