linterjs / core

Lint all the things... with one command
MIT License
4 stars 0 forks source link

Use and report to codecov #30

Closed Shinigami92 closed 4 years ago

Shinigami92 commented 4 years ago

closes #24

This PR is based on https://github.com/typescript-eslint/typescript-eslint

codecov[bot] commented 4 years ago

Codecov Report

Merging #30 into master will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #30   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines          109       122   +13     
  Branches        19        25    +6     
=========================================
+ Hits           109       122   +13     
Flag Coverage Δ
#unittest 100.00% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/format.ts 100.00% <ø> (ø)
src/lint.ts 100.00% <ø> (ø)
src/linter-provider.ts 100.00% <100.00%> (ø)
src/linter.ts 100.00% <100.00%> (ø)
src/logger.ts 100.00% <100.00%> (ø)
zimme commented 4 years ago

I'm thinking that we might skip codecov and just use codeclimate. https://docs.codeclimate.com/docs/github-actions-test-coverage

Then we'll get both code coverage and some maintainability score, suggestions.

Shinigami92 commented 4 years ago

superseded by #36