New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(postcss-colormin): Fix bug where color value is mutated #546
Conversation
…sforming to hsl Fixes cssnano#545
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add tests?
@evilebottnawi I've update the existing test case for rgba to hsla to cover that use-case plus updated the expected result of two others that change along with this fix. |
@weaintplastic thanks 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The integration tests need to be updated as well.
@@ -6,7 +6,7 @@ import { | |||
processCSSWithPresetFactory, | |||
} from '../../../../util/testHelpers'; | |||
|
|||
const {passthroughCSS, processCSS} = processCSSFactory(plugin); | |||
const { passthroughCSS, processCSS } = processCSSFactory(plugin); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is causing issues with our linting settings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andyjansson Sorry, I've missed that and fixed it with the next commit.
@andyjansson integration tests have been updated and CI check is green. Thanks for your direction. 🙌🏻 |
/cc @andyjansson |
@andyjansson looks good for me, merge, feel free to feedback |
Fix bug where color value is mutated when tranforming to hsl
Fixes #545