no-magic-numbers should ignore various Number methods' params

[mateuszwitkowski]

Bug Report

Since linter now ingores radix param in parseInt method maybe it should be good to standardize how params are handled in those Number methods:

• toFixed()
• toExponential()
• toPrecision()

TypeScript code being linted

someNumber.toFixed(2);
someNumber.toExponential(4);
someNumber.toPrecision(2);

with tslint.json configuration:

{
  "no-magic-number": true
}

Actual behavior

[tslint] 'magic numbers' are not allowed (no-magic-numbers)

Expected behavior

No error.

[mateuszwitkowski]

If it's OK I'd like to take care of this but I need some hint: how to get the name of called method?

[aervin]

@mateuszwitkowski You should paste these snippets

someNumber.toFixed(2);
someNumber.toExponential(4);
someNumber.toPrecision(2);

into ast-explorer to get a better idea of the AST nodes you'll be parsing.

[mateuszwitkowski]

@aervin That's great tool. Thanks!

[JoshuaKGoldberg]

This was fixed by #3668 🎉