Skip to content
New issue

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

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

Already on GitHub? # to your account

Add additional eslint rules. #825

Merged
merged 1 commit into from
Apr 2, 2018
Merged

Conversation

mrstegeman
Copy link
Contributor

No description provided.

@mrstegeman mrstegeman requested a review from hobinjk April 2, 2018 20:46
@ghost ghost assigned mrstegeman Apr 2, 2018
@ghost ghost added the review label Apr 2, 2018
@codecov-io
Copy link

Codecov Report

Merging #825 into master will decrease coverage by 0.04%.
The diff coverage is 69.86%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #825      +/-   ##
==========================================
- Coverage   77.19%   77.14%   -0.05%     
==========================================
  Files          87       87              
  Lines        3762     3763       +1     
  Branches      501      501              
==========================================
- Hits         2904     2903       -1     
- Misses        787      788       +1     
- Partials       71       72       +1
Impacted Files Coverage Δ
src/rules-engine/effects/SetEffect.js 100% <ø> (ø) ⬆️
src/controllers/logs_controller.js 96.42% <ø> (ø) ⬆️
src/plugin/device-proxy.js 75% <0%> (ø) ⬆️
src/controllers/new_things_controller.js 43.75% <0%> (ø) ⬆️
src/rules-engine/triggers/TimeTrigger.js 13.63% <0%> (ø) ⬆️
src/rules-engine/effects/PulseEffect.js 100% <100%> (ø) ⬆️
src/rules-engine/APIError.js 100% <100%> (ø) ⬆️
src/router.js 97.91% <100%> (ø) ⬆️
src/models/user.js 100% <100%> (ø) ⬆️
src/jwt-middleware.js 85.45% <100%> (ø) ⬆️
... and 48 more

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 5f2dff9...d60ee5c. Read the comment docs.

@mrstegeman mrstegeman merged commit bce215b into WebThingsIO:master Apr 2, 2018
@ghost ghost removed the review label Apr 2, 2018
@mrstegeman mrstegeman deleted the eslint branch April 2, 2018 20:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants