Skip to content
This repository has been archived by the owner on May 31, 2021. It is now read-only.

Enable console #140

Merged
merged 2 commits into from
Dec 5, 2019
Merged

Enable console #140

merged 2 commits into from
Dec 5, 2019

Conversation

thislooksfun
Copy link
Owner

No description provided.

@todo
Copy link

todo bot commented Dec 5, 2019

Disable console in production.

earthdawn/.eslintrc.js

Lines 8 to 13 in 8536335

// TODO: Disable console in production.
// "no-console": process.env.NODE_ENV === "production" ? "error" : "off",
"no-console": "off",
"no-debugger": process.env.NODE_ENV === "production" ? "error" : "off",
},
parserOptions: {


This comment was generated by todo based on a TODO comment in 8536335 in #140. cc @thislooksfun.

@github-actions github-actions bot added the style Changes to the style guides label Dec 5, 2019
@thislooksfun thislooksfun added the ready-to-merge This PR will be merge automatically once all checks pass. label Dec 5, 2019
@codecov-io
Copy link

codecov-io commented Dec 5, 2019

Codecov Report

Merging #140 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #140   +/-   ##
=======================================
  Coverage   21.22%   21.22%           
=======================================
  Files          51       51           
  Lines         636      636           
  Branches       49       49           
=======================================
  Hits          135      135           
  Misses        501      501
Flag Coverage Δ
#unittests 21.22% <ø> (ø) ⬆️

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 aed19c3...a6ded17. Read the comment docs.

@mergify mergify bot merged commit 9c5ca65 into master Dec 5, 2019
@mergify mergify bot deleted the feature/auto-build branch December 5, 2019 05:24
@todo todo bot mentioned this pull request Dec 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready-to-merge This PR will be merge automatically once all checks pass. style Changes to the style guides
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants