newtmitch / docker-sonar-scanner

Quick sonar scanner docker image
MIT License
129 stars 88 forks source link

Scanner properties overrides server settings #25

Closed theoutsider24 closed 5 years ago

theoutsider24 commented 5 years ago

The line to exclude node modules from the scan ("sonar.exclusions=/node_modules//*") means that server-side configuration of ignored files will never be used since there's an existing scanner-level property

newtmitch commented 5 years ago

Good to know. I'll update the README to make that fact clear.

newtmitch commented 5 years ago

Ahh, nevermind, I see my comment in the README here:

I've added this to the deafult sonar-runner.properties file, so remove that if you don't want it there for some reason.

I'll take a look at that.

newtmitch commented 5 years ago

Fixed. Thanks for the report @theoutsider24