Closed armano2 closed 5 years ago
Merging #39 into master will increase coverage by
0.86%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #39 +/- ##
==========================================
+ Coverage 92.68% 93.54% +0.86%
==========================================
Files 5 6 +1
Lines 82 93 +11
Branches 19 23 +4
==========================================
+ Hits 76 87 +11
Misses 6 6
Impacted Files | Coverage Δ | |
---|---|---|
src/package-name.ts | 100% <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 03e6da0...6adb56a. Read the comment docs.
Do I have to do something here¿
Do I have to do something here¿
Nope, just didn’t have a chance to review this. Sorry for the delay!
can i merge this?
this PR contains function extracted from eslint: https://github.com/eslint/eslint/blob/6009239042cb651bc7ca6b8c81bbe44c40327430/lib/util/naming.js#L69
its best if its going to use same logic to generate package name as eslint to limit potential issues with other packages
fixes: #38