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 latest version of purgecss plugin #48

Closed
wants to merge 3 commits into from

Conversation

chood531
Copy link

I am trying to get this issue pulled into my project so would like to update the purgecss plugin version: FullHuman/purgecss#92

I am trying to get this issue pulled into my project so would like to update the purgecss plugin version: FullHuman/purgecss#92
@freekmurze
Copy link
Member

Tests seem to be failing for this one.

@chood531
Copy link
Author

@freekmurze - unfortunately, the tests are also failing on master.

I tried to fix, but can't seem to figure it out. Seems to be something with Jest.

I updated the version of Jest to fix one issue (jestjs/jest#6766 (comment)), but that introduced another issue.

Sorry I couldn't help try to fix it. Couldn't seem to figure it out on my end :(

@freekmurze
Copy link
Member

No worries, we'll take a look at this soon.

@sebastiandedeyne
Copy link
Member

I'll look into this soon. Gonna hold off merging a major version update of Purgecss until we've ensured it all works as intended.

@sebastiandedeyne
Copy link
Member

Ended up merging #49 for this because it solved the test issues, thanks for your time either way!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants