ryanluker / vscode-coverage-gutters

Display test coverage generated by lcov and xml - works with many languages
https://marketplace.visualstudio.com/items?itemName=ryanluker.vscode-coverage-gutters
MIT License
460 stars 88 forks source link

Issue 367 / Switch out tslint for eslint #368

Closed timagixe closed 2 years ago

timagixe commented 2 years ago

Closes #367

ryanluker commented 2 years ago

@timagixe Looking great so far, thanks for tackling this! I have approved the PR to run against the CI system and also provided a few bits of feedback above.

(we might also need to update the npm scripts to include a few different eslint options based on the CI output🤔) https://github.com/ryanluker/vscode-coverage-gutters/runs/6274012413?check_suite_focus=true https://eslint.org/docs/user-guide/command-line-interface#--ext

timagixe commented 2 years ago

@ryanluker thanks for running CI, I have adjusted scripts in https://github.com/ryanluker/vscode-coverage-gutters/pull/368/commits/d00386bfd9f9f187581e9e8da3e818431b518efa. I hope this time all the checks will fly 🚀