Upgrde SonarQube to 7.8 and sonar-scala to 7.6. #14
Conversation
| @@ -15,6 +15,13 @@ | |||
| }, | |||
| "versions": { | |||
| "current": [ | |||
There was a problem hiding this comment.
Why does this list say current, if it's in reality contains old/previous versions?
There was a problem hiding this comment.
all it means really is the non-lts versions, perhaps that's exactly what I should have named it
| @@ -0,0 +1,12 @@ | |||
| # SonarQube 7.8 image with bundled sonar-scala 7.6.0 (https://github.com/mwz/sonar-scala). | |||
|
|
|||
| FROM sonarqube:7.8-community | |||
There was a problem hiding this comment.
Are you sure you want to keep a separate folder for every old version you had, despite it's outdated?
Would it be better just to change one Dockerfile upgrading some versions?
There was a problem hiding this comment.
Yeah, I was planning to clean up this repo, so I'll do that at some point.
|
Btw, how about updating to They claim that they added Scala support in this release. Can we use it instead of |
There was a problem hiding this comment.
Btw, how about updating to
7.9 LTS, since it's already out there?
I haven't upgraded sonar-scala to the latest LTS yet, but I'll get around doing that soon.
They claim that they added Scala support in this release. Can we use it instead of
scapegoat,scoverageandscalastyle?
They have their own Scala plugin called SonarScala while I maintain the community version sonar-scala which comes with a slightly different feature set and works differently from the official plugin. Have a look at the comparison here to get a better idea about the differences.
| @@ -0,0 +1,12 @@ | |||
| # SonarQube 7.8 image with bundled sonar-scala 7.6.0 (https://github.com/mwz/sonar-scala). | |||
|
|
|||
| FROM sonarqube:7.8-community | |||
There was a problem hiding this comment.
Yeah, I was planning to clean up this repo, so I'll do that at some point.
| @@ -15,6 +15,13 @@ | |||
| }, | |||
| "versions": { | |||
| "current": [ | |||
There was a problem hiding this comment.
all it means really is the non-lts versions, perhaps that's exactly what I should have named it
I also changed the following: