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

Bump gradle version to 8.4 #39

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Bump gradle version to 8.4 #39

merged 1 commit into from
Nov 21, 2023

Conversation

marcodelapierre
Copy link
Contributor

@marcodelapierre marcodelapierre commented Nov 21, 2023

This PR will upgrade gradle to 8.4

@marcodelapierre marcodelapierre changed the title Bump gradle version Bump gradle version to 8.4 Nov 21, 2023
@pditommaso pditommaso merged commit 399f0b2 into master Nov 21, 2023
@pditommaso pditommaso deleted the update/gradle_version branch November 21, 2023 08:01
@marcodelapierre marcodelapierre self-assigned this Nov 21, 2023
@marcodelapierre
Copy link
Contributor Author

From Munish:

@munishchouhan
Copy link
Member

PR with fix
#40

@pditommaso
Copy link
Contributor

Why tests were skipped by the CI?

@marcodelapierre
Copy link
Contributor Author

I remember this has happened before with another PR of mine.

One thing I can think of is that, as the change was little, I edited directly in the github web interface, then asked to commit on new branch, which I then submitted for PR .. wondering whether this workflow works in a different way compared to a push from local repo .. it shouldn't right?

Can investigate further

@marcodelapierre
Copy link
Contributor Author

See #44

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.

3 participants