Skip to content
This repository has been archived by the owner on Mar 14, 2022. It is now read-only.

patch: Do not error on JS lint warnings

Compare
Choose a tag to compare
@origamiserviceuser origamiserviceuser released this 11 Jun 19:53
· 208 commits to master since this release
111b160
Do not error on JS lint warnings (#807)

* Do not error on JS lint warnings

The esline upgrade has caused errors to throw on lint warnings.
This commit only throws an error if there is a lint error, and
logs out any warnings.

See this o-typography build which previously passed:
https://github.com/Financial-Times/o-typography/runs/762488094

* Update test/unit/tasks/verify-javascript.test.js

Co-authored-by: Jake Champion <[email protected]>

* Apply suggestions from code review

Co-authored-by: Jake Champion <[email protected]>