Skip to content

Conversation

@mmarchini
Copy link
Contributor

Disable lint check by default and make it opt-in, so it doesn't impact
the workflow for collaborators, while providing a way for early birds to
try it out so we can tweak it until it works great for everyone (at
which point we can re-enable it by default).

Ref: #484

@codecov
Copy link

codecovbot commented Aug 20, 2020

Codecov Report

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

Impacted file tree graph

@@ Coverage Diff @@## master #487 +/- ## ======================================= Coverage 82.59% 82.59% ======================================= Files 34 34 Lines 1660 1660 ======================================= Hits 1371 1371 Misses 289 289 

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 17ea885...595c4f8. Read the comment docs.

Disable lint check by default and make it opt-in, so it doesn't impact the workflow for collaborators, while providing a way for early birds to try it out so we can tweak it until it works great for everyone (at which point we can re-enable it by default). Ref: nodejs#484
@mmarchinimmarchini merged commit b567c1e into nodejs:masterAug 20, 2020
@mmarchinimmarchini deleted the opt-in-lint branch August 20, 2020 16:19
Sign up for freeto 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.

2 participants

@mmarchini@codebytere