Skip to content

Normalize numbers in CSS like JS#2627

Merged
lydell merged 2 commits intoprettier:masterfrom
lydell:css-numbers
Aug 22, 2017
Merged

Normalize numbers in CSS like JS#2627
lydell merged 2 commits intoprettier:masterfrom
lydell:css-numbers

Conversation

@lydell
Copy link
Copy Markdown
Member

@lydell lydell commented Aug 15, 2017

Unlike JS, 1.0 is also normalized to 1.

@lydell
Copy link
Copy Markdown
Member Author

lydell commented Aug 16, 2017

Yay, the tests passed when I switched postcss-values-parser to shellscape/postcss-values-parser#36

Do we want to wait for shellscape/postcss-values-parser#36 to be merged first?

@vjeux
Copy link
Copy Markdown
Contributor

vjeux commented Aug 16, 2017

Woow, thanks for doing that :) It would be better if it was merged but if it takes too long to get it reviewed, feel free to hit the merge button :)

@vjeux
Copy link
Copy Markdown
Contributor

vjeux commented Aug 22, 2017

Looks like the other PR is not being reviewed, so feel free to merge.

@lydell lydell merged commit 854bb3c into prettier:master Aug 22, 2017
@lydell lydell deleted the css-numbers branch August 22, 2017 17:05
@gaganjakhotiya
Copy link
Copy Markdown

gaganjakhotiya commented Sep 26, 2017

@lydell the updated SHA af2c80b2bb558a6e7d61540d97f068f9fa162b38 of postcss-values-parser repo doesn't seem to exist. I'm unable to setup the pre-commit hook because of this.

@lydell
Copy link
Copy Markdown
Member Author

lydell commented Sep 27, 2017

@gaganjakhotiya Oops! I accidentally deleted the branch when my PR was merged :( It should be back now. Good catch!

@lock lock Bot added the locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting. label Jan 19, 2019
@lock lock Bot locked as resolved and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants