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

Add Gradle 8 Test Build #287

Merged
merged 10 commits into from
Jul 21, 2023
Merged

Add Gradle 8 Test Build #287

merged 10 commits into from
Jul 21, 2023

Conversation

f4lco
Copy link
Collaborator

@f4lco f4lco commented Jul 13, 2023

No description provided.

@f4lco f4lco marked this pull request as ready for review July 21, 2023 07:09
Signed-off-by: Falco Duersch <github@falcoduersch.com>
@f4lco f4lco merged commit d289588 into master Jul 21, 2023
3 checks passed
@f4lco f4lco deleted the feature/gradle8 branch July 21, 2023 07:27
integrationTests/buildSrc/build.gradle Show resolved Hide resolved
@@ -3,7 +3,7 @@ geckoDriverVersion = 0.29.1
geckoDriverPlatform = macos
gradle_download_task_version = 3.4.3
commons_configuration_version = 1.10
gebVersion=2.3.1
gebVersion=5.0
Copy link
Member

Choose a reason for hiding this comment

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

I saw you downgraded Geb's version from 5.1 to 5.0 (above in the GH Actions script - but I changed it so that's why I'm asking here). What was the problem?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ah, very very good catch. I accidentally undid my own dep upgrades from 126fcc3 when copy-and-pasting the CI workflow from Gretty 3.
I'll give it a new try, actually we went back on both, geb and spock.

So it's probably spock_version = 2.3-groovy-3.0 and gebVersion = 5.1 to get back to the state before this PR (without further upgrading).

f4lco added a commit that referenced this pull request Jul 21, 2023
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