-
Notifications
You must be signed in to change notification settings - Fork 52
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
Fix sonar scanner errors due to Java deprecated version #265
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information The version of Java (11.0.14.1) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17. |
7ecc792
to
d9f6fe5
Compare
There is a also new way (not sure if better) to make this type of solutions in this style here
As an future option, we can move to that and use the |
Due to end of support of the Java 11 LTS in Sonar, this is needed to continue using the software. The JRE included in the sonar-scanner is used to keep the version of Java current, and that is the reason for a newer version. Node.js version was updated to suppress the warning for a future end of support. Signed-off-by: Leonardo Carreras <[email protected]>
fa4111f
to
e3100d9
Compare
All commits were squashed to avoid intermediate non-working solutions going to master branch |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Due to end of support of the Java 11 (LTS) in Sonar, this PR updates the version of Sonar Scanner to 5.0.1.3006. Additionally, the warnings because of Node.js (version 16 to be unsupported in the near future) are corrected. Actions
cache
is also updated tov4
. The action forsetup-java
was removed (we use now the JRE included insonar-scanner
).This closes #251