Bypass TS6133 error by allowing function declarations that are unused

[OmarDarwish]

With the release of TypeScript 2.6.1, we now have the option to suppress errors in .ts files using '// @ts-ignore' comments.

Fixes #322

[SupernaviX]

This is pointing to an internal registry

[OmarDarwish]

Resolved. Thanks!

[OmarDarwish]

Here's a benchmark run and test suites for these changes:

bash-3.2$ npm run benchmark && npm test
npm info it worked if it ends with ok
npm info using [email protected]
npm info using [email protected]
npm info lifecycle [email protected]~prebenchmark: [email protected]
npm info lifecycle [email protected]~benchmark: [email protected]

> [email protected] benchmark /Users/darwisho/code/nearley
> benchr test/benchmark.js

• test/benchmark.js:

  • calculator.

    ✔  nearley  13,480.40  ops/sec  ±0.88%  (87 runs)

  • json.

    ✔  nearley  110.40  ops/sec  ±4.29%  (73 runs)

  • tosh.

    ✔  nearley  2,179.87  ops/sec  ±0.89%  (87 runs)

npm info lifecycle [email protected]~postbenchmark: [email protected]
npm info ok
npm info it worked if it ends with ok
npm info using [email protected]
npm info using [email protected]
npm info lifecycle [email protected]~pretest: [email protected]
npm info lifecycle [email protected]~test: [email protected]

> [email protected] test /Users/darwisho/code/nearley
> mocha test/*.test.js



  Linter
    ✓ runs without warnings on empty rules
    ✓ warns about undefined symbol
    ✓ doesn't warn about defined symbol
    ✓ doesn't warn about duplicate symbol

  bin/nearleyc
    ✓ builds for ES5 (295ms)
    ✓ builds for ES6+ (1566ms)
    ✓ builds for CoffeeScript (614ms)
    ✓ builds for TypeScript (1854ms)
    ✓ builds modules in folders (297ms)
    ✓ builds modules with multiple includes of the same file (289ms)
    ✓ warns about undefined symbol (295ms)
    ✓ doesn't warn when used with the --quiet option (283ms)

  nearleyc: example grammars
    ✓ calculator example
    ✓ csscolor example
    ✓ exponential whitespace bug
    ✓ percent bug
    ✓ json
    ✓ classic crontab (54ms)
    ✓ case-insensitive strings

  nearleyc: builtins
    ✓ generate includes id
    - nuller

  nearleyc: macros
    ✓ seems to work
    ✓ must be defined before use
    ✓ compiles a simple macro from external file

  Parser: API
    ✓ shows line number in errors
    ✓ shows token index in errors
    ✓ can save state
    ✓ can rewind
    ✓ won't rewind without `keepHistory` option
    ✓ restores line numbers
    ✓ restores column number

  Parser: examples
    ✓ nullable whitespace bug
    ✓ parentheses
    ✓ tokens
    ✓ json
    ✓ tosh (227ms)


  35 passing (6s)
  1 pending

npm info lifecycle [email protected]~posttest: [email protected]
npm info ok

[tjvr]

Looks good, thank you :)

[OmarDarwish]

Thanks @tjvr. What's the npm release cadence for Nearley?

[tjvr]

Um, whenever someone bugs me to do a release, I suppose? :)

[OmarDarwish]

@tjvr I am officially bugging you to do a release 😁