-
Notifications
You must be signed in to change notification settings - Fork 54
Update versions #35
base: master
Are you sure you want to change the base?
Update versions #35
Conversation
d80689c
to
cd97e3d
Compare
@roganw: Please, comment on this. |
Hi ! Sorry to highjack a bit your PR. As stated in #36, we're using an EOL nodejs version for this runner. As you're updating some dependancies/runtimes on this image, could you check nodejs version in your new image, and if possible install latest LTS (v16) as stated on official guide (https://github.com/nodesource/distributions/blob/master/README.md#installation-instructions) ? Thanks ! |
Hi @Elanis! I used openjdk:11.0.14.1-jre image for the plugin, which is based on Debian 11 (bullseye). The nodejs package is installed then from that distribution and has version 12.22.5. If we need to run latest LTS of nodejs we should use some other base image or more recent distribution. I have no plans upgrading Debian 11 in openjdk base image, so the nodejs package version will be 12.22.5 for now. UPD: We can switch to sonarsource/sonar-scanner-cli:4.6 base image, where Alpine 3.14 is used as a base. There is nodejs version 14.19.0 there. Maybe this will be the best option. |
Okay, thanks for checking :) NodeJS 12 has a EOL a year later than v10 (2022-04-30 instead of 2021-04-30). That's better than the previous one :) |
@Elanis I updated the PR to use sonarsource/sonar-scanner-cli:4.6 base image. |
@roganw Can this be reviewed and merged? |
1 similar comment
@roganw Can this be reviewed and merged? |
No description provided.