ath0mas commented 3 years ago

Platforms affected

Docs

Motivation and Context

The command to lint described in Readme file does not follow the changes introduced in #822.

Description

Correct with "lint" NPM script like in package.json.

Testing

.

Checklist

[x] I've run the tests to see all new and existing tests pass
[x] I added automated test coverage as appropriate for this change
[x] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[x] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[x] I've updated the documentation if necessary

codecov-commenter commented 3 years ago

Codecov Report

Merging #877 (4e71f55) into master (3274a78) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 4e71f55 differs from pull request most recent head d68b8d0. Consider uploading reports for the commit d68b8d0 to get more accurate results

@@           Coverage Diff           @@
##           master     #877   +/-   ##
=======================================
  Coverage   91.13%   91.13%           
=======================================
  Files          45       45           
  Lines        2053     2053           
=======================================
  Hits         1871     1871           
  Misses        182      182

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 3274a78...d68b8d0. Read the comment docs.

raphinesse commented 3 years ago

Thanks for fixing this! :tada:

apache / cordova-lib

docs: correct linter command in Readme #877