scapegoat

scapegoat copied to clipboard

Updates [org.scoverage:sbt-scoverage](https://github.com/scoverage/sbt-scoverage) from 1.9.3 to 2.0.2. [GitHub Release Notes](https://github.com/scoverage/sbt-scoverage/releases/tag/v2.0.2) - [Version Diff](https://github.com/scoverage/sbt-scoverage/compare/v1.9.3...v2.0.2) I'll automatically update this PR to resolve conflicts as long as you don't change it yourself. If you'd...

scala-steward

New inspection level - SKIP

3

Current [inspection levels](https://github.com/scapegoat-scala/scapegoat/blob/master/src/main/scala/com/sksamuel/scapegoat/Level.scala) are: `ERROR`, `WARNING`, `INFO`. I suggest to add a new one: (working name) `SKIP`. So that for instance one can set it as a default level for...

mccartney

Please compile support for Scala 3.0.0

14

mr-git

false positive VariableShadowing when use copy in map

1

Bellow code generate VariableShadowing warning. ``` [scapegoat] [VariableShadowing] Variable shadowing [warn] Variable shadowing is very useful, but can easily lead to nasty bugs in your code. Shadowed variables can be...

btomala

Looking for new maintainer

5

I am currently spread too thin between many open source projects, and since I don't currently use Scala, this project would benefit from someone owning it full time. If anyone...

sksamuel

When a triple quoted string contains "$" characters, Scapegoat produces a false positive warning for "Looks like interpolated String". The use of triple quote Strings in Scala to pass query...

ian-welsh

PreferSetEmpty,etc. seem misleading

3

The inspection currently states ``` "`Set[T]()` allocates an intermediate object. Consider `Set.empty` which returns a singleton instance without creating a new object." ``` Arguably the `apply()` method does not document...

gawashburn

False-positive warnings for checks: Unused parameter, Null assignment, Variable shadowing, While true loop, Empty if expression

3

I use scapegoat in project _akka/akka_, and find more than 90% of warnings from this five checks are not what I need. **1. Unused parameter** when the code is like:...

Stardust1225

It appears that backtick characters (used for inline code blocks in explanation and description fields of inspections) are leaking to console and HTML report output and we should strip them.

mwz

Results of running Scapegoat against Scapegoat

6

I've run the current Scapegoat snapshot against the codebase and you can find the full report below. It would be good to fix some of those issues and also set...

mwz