tidev / titanium-sdk

🚀 Native iOS and Android Apps with JavaScript
https://titaniumsdk.com/
Other
2.76k stars 1.21k forks source link

chore: java/node version update #14104

Closed m1ga closed 2 months ago

m1ga commented 2 months ago

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).

cb1kenobi commented 2 months ago

I wish this was easiser. Thank you for trying!

m1ga commented 2 months ago

sorry if you get notifications all the time :blush: 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 commented 2 months ago

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.