newtmitch/docker-sonar-scanner

add node.js to alpine image

Closed this issue · 6 comments

could node.js as well be added to the alpine-based image? currently it's only included in the full version.

To support JS/TS scanning.

follow up to #9

Thanks for the spot on this, @mobidev111 - I've added it to the apk add command and it looks like it "took" okay, but I don't have time to test this for a little while. If I push this up to a branch would you be able to pull that down and make sure it solves the problem for you?

Yes, I can try this out.

could you publish the fix on the dockerhub? @newtmitch

I can do that. Was waiting to hear from @mobidev111 to see if it worked out right first, but might go ahead and push anyway if I don't hear back. 😄

@pawelmarkowski pushed. let me know if any issues.

@newtmitch works great! Thanks