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 to latest gradle version and add gradle wrapper #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Joseph5610
Copy link

Hello,

While we tried to use the allure-server tool, we discovered that it is build with rather old version of gradle.
This PR updates the gradle version to 8.7. With that, the gradle docker image in the Dockerfile is updated as well as the github actions used to build the project.

For some specific reason, this repo excluded the gradle wrapper from the VCS. While updating the gradle github action, I found out that the wrapper is now the preferred way how to work with it. For this reason, I removed required wrapper files from gitignore. Even the gradle team itself recommends that the wrapper files are so small so they should be committed into the repo - https://docs.gradle.org/current/userguide/gradle_wrapper.html#sec:adding_wrapper so I don´t expect it will cause any problems, on the contrary, the presence of gradle wrapper files in the VCS can simplify the first load of the project.

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

1 participant