Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Backport features from 7.x #136

Merged
merged 6 commits into from Dec 8, 2018
Merged

Backport features from 7.x #136

merged 6 commits into from Dec 8, 2018

Conversation

mwz
Copy link
Member

@mwz mwz commented Dec 4, 2018

Backporting features from 7.x - includes the recommended quality profile, updated generation of Scapegoat rules, version bumps and other minor tweaks.

@@ -1 +1 @@
sbt.version = 1.2.3
sbt.version = 1.2.6
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are already in sbt 1.2.7.

@@ -42,6 +42,6 @@ private[scapegoat] final case class ScapegoatInspection(
defaultLevel: Level
)

private[scapegoat] object ScapegoatInspections {
object ScapegoatInspections {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why made it public ?
With the work you did in #130 this can be _private to the scapegoat package, the same can go with the ScalastyleInspections object.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like I missed it after refactoring.

Copy link
Contributor

@BalmungSan BalmungSan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Except for the detail with the ScapegoatInspections object, this looks great.

@mwz mwz merged commit 94ddc77 into sonar-6.7-lts Dec 8, 2018
@mwz mwz deleted the backport-from-7.x branch December 8, 2018 00:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants