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

Handle empty path object in browser builds #1538

Merged
merged 2 commits into from
Mar 9, 2021

Conversation

mattcompiles
Copy link
Contributor

I've also fixed the browser tests as they were not mocking correctly. Calling doMock after importing postcss had no effect as the module had already been processed before the mock takes effect.

@ai
Copy link
Member

ai commented Mar 9, 2021

Looks good. Can you just call npx eslint --fix . to fix code format?

@mattcompiles
Copy link
Contributor Author

Looks good. Can you just call npx eslint --fix . to fix code format?

👍

@ai ai merged commit 81704c9 into postcss:main Mar 9, 2021
@ai
Copy link
Member

ai commented Mar 9, 2021

Thanks. Released in 8.2.8.

This was referenced Mar 9, 2021
This was referenced Mar 15, 2021
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

2 participants