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

chore: java/node version update #14104

Closed
wants to merge 7 commits into from
Closed

chore: java/node version update #14104

wants to merge 7 commits into from

Conversation

m1ga
Copy link
Contributor

@m1ga m1ga commented Sep 5, 2024

Testing if I can remove the

 Not a number: 33x:
java.lang.NumberFormatException: Not a number: 33x
	at com.sun.xml.bind.DatatypeConverterImpl._parseInt(DatatypeConverterImpl.java:95)

warnings. I don't have them when I build locally.

Updating various version in the github action to see if anything helps. It still builds the SDK and I don't see them locally (Fedora).

@m1ga m1ga closed this Sep 5, 2024
@m1ga m1ga deleted the choreJavaVersion branch September 5, 2024 22:10
@cb1kenobi
Copy link
Contributor

I wish this was easiser. Thank you for trying!

@m1ga
Copy link
Contributor Author

m1ga commented Sep 5, 2024

sorry if you get notifications all the time 😊 It's strange that it is only happening online and not locally. It looks like it is gradle, so the other PR that raises gradle should fix it.

side not: we can update java, node and ubuntu at one point. That was working fine

@cb1kenobi
Copy link
Contributor

Notifications are fine. :)

Ideally we could/should put the gradle version behind a feature flag, the users can opt-in until the next major where we flip the default. I understand this takes a bit of effort and I have been stalling. Sigh.

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