Closed EricRovell closed 3 years ago
Oh, wow! Amazing job! 🔥 Thanks a lot for that.
The code looks great but there is a few eslint warning that we need to fix before merge. Please run npm run lint
in the project root to lint the code.
Got it!
Merging #58 (e8a589c) into master (c0bdc02) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #58 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 35 37 +2
Lines 441 482 +41
Branches 96 108 +12
=========================================
+ Hits 441 482 +41
Impacted Files | Coverage Δ | |
---|---|---|
src/colorModels/cmyk.ts | 100.00% <100.00%> (ø) |
|
src/plugins/cmyk.ts | 100.00% <100.00%> (ø) |
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 c0bdc02...e8a589c. Read the comment docs.
You are welcome, I will be back with more :)
Implemented the CMYK plugin. It adds the functionality to convert the color to CMYK color model object, convert to CMYK color string and parsing the CMYK color objects. Updated the documentation . Implemented and ran the tests.