Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tslint emit failures as errors #32

Merged
merged 2 commits into from
Sep 15, 2017
Merged

tslint emit failures as errors #32

merged 2 commits into from
Sep 15, 2017

Conversation

zinserjan
Copy link
Owner

Introduces option tslintEmitErrors to override the defined severity for each rule and emit failures always as error.

resolves #27

@codecov-io
Copy link

codecov-io commented Sep 15, 2017

Codecov Report

Merging #32 into master will increase coverage by 0.11%.
The diff coverage is 45.45%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
+ Coverage   40.29%   40.41%   +0.11%     
==========================================
  Files           8        8              
  Lines         335      339       +4     
  Branches       39       44       +5     
==========================================
+ Hits          135      137       +2     
- Misses        200      202       +2
Impacted Files Coverage Δ
src/worker/TsCheckerRuntime.ts 0% <0%> (ø) ⬆️
src/checker/resultSerializer.ts 50% <0%> (-6.25%) ⬇️
src/TsCheckerWebpackPlugin.ts 96.15% <100%> (+0.1%) ⬆️
src/worker/TsCheckerWorker.ts 90.24% <100%> (ø) ⬆️

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 aad9653...77d9891. Read the comment docs.

@zinserjan zinserjan force-pushed the tslint-emit-errors branch 2 times, most recently from b02b350 to 2aaa38a Compare September 15, 2017 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow configuring tslint to fail on warnings
2 participants