sprintf.js icon indicating copy to clipboard operation
sprintf.js copied to clipboard

Tests code improvements

Open rastorc3v opened this issue 2 years ago • 1 comments

  • var replaced with const

  • anonymous functions replaced with arrow functions

  • PI value replaced with Math.PI constant

  • equal replaced with strictEqual since the first one was deprecated

  • [x] tests

  • [x] linter

rastorc3v avatar Feb 02 '23 22:02 rastorc3v

equal replaced with strictEqual since the first one was deprecated

The better way to do this is by loading assert like this: const assert = require('assert/strict'), or if we're willing to drop support for older versions of node: const assert = require('node:assert/strict').

msimerson avatar Apr 05 '24 02:04 msimerson