Skip to content
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

Update to use postcss and cssnano for SeparateKeyframes transform #679

Merged
merged 5 commits into from
Apr 1, 2020

Conversation

ijjk
Copy link
Contributor

@ijjk ijjk commented Apr 1, 2020

This updates the SeparateKeyframes transform to leverage postcss instead of the css package. It also adds cssnano to provide better minification when enabled.

@sebastianbenz
Copy link
Collaborator

Thanks for doing this. Can you please rebase against master, this should fix the build error.

@ijjk
Copy link
Contributor Author

ijjk commented Apr 1, 2020

@sebastianbenz I double checked and it looks like this branch is caught up with master correctly. The failing test appears to also be failing for master right now. Let me know if I'm missing something and I can update this branch 👍

@sebastianbenz
Copy link
Collaborator

My mistake, fix was not yet on master (but is now).

Copy link
Collaborator

@sebastianbenz sebastianbenz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@sebastianbenz sebastianbenz merged commit 2f273cf into ampproject:master Apr 1, 2020
@ijjk ijjk deleted the update/css-processing branch April 1, 2020 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants