Ignore magic number for named argument

[gmarti]

Magic number should probably not be reported when using named argument

Please let me know if something is wrong, first PR here.

Added tests too

[codecov-io]

Codecov Report

Merging #290 into master will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff          @@
##           master   #290   +/-   ##
=====================================
  Coverage       0%     0%           
=====================================
  Files          62     62           
  Lines        1495   1503    +8     
  Branches      146    143    -3     
=====================================
- Misses       1495   1503    +8

Impacted Files	Coverage Δ
...rg/scalastyle/scalariform/MagicNumberChecker.scala	0% <0%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8623a18...be6c773. Read the comment docs.

[matthewfarwell]

Sorry I've been so long in responding to this, this slipped underneath my radar.

I would love this as a PR, but I think it should be an option to the checker. Maybe something like allowNamedArguments, which would be a boolean, default false?

[gmarti]

Thx for taking the time to check this PR. I have added the option to the checker defaulted to false.

[gmarti]

Any news ?

[weixu365]

Any news on this PR? We are using lots of named parameters, and have to struggle with the magic number rule for now. It would be awesome if this could be merged

[gmarti]

@matthewfarwell ?