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

chore(deps): upgrade postcss-modules #310

Closed
wants to merge 3 commits into from

Conversation

AndrewLeedham
Copy link

Enables exportGlobals option. I also added a test, to make sure this works as expected.

Enables exportGlobals option
@codecov
Copy link

codecov bot commented Aug 18, 2020

Codecov Report

Merging #310 into master will decrease coverage by 0.18%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #310      +/-   ##
==========================================
- Coverage   92.68%   92.50%   -0.19%     
==========================================
  Files          10       10              
  Lines         328      320       -8     
  Branches      115      113       -2     
==========================================
- Hits          304      296       -8     
  Misses         23       23              
  Partials        1        1              
Impacted Files Coverage Δ
src/index.js 98.97% <0.00%> (-0.08%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3de028d...075cf43. Read the comment docs.

@tjenkinson
Copy link

Please can you bump this to 3.2.1?

This includes this fix (madyankin/postcss-modules#111) which we need.

pascalduez added a commit to pascalduez/react-module-boilerplate that referenced this pull request Aug 20, 2020
- Fix duplicated rules
- Temporary until egoist/rollup-plugin-postcss#310 is published
@wardpeet
Copy link
Collaborator

@egoist It seems like there are no breaking changes in https://github.com/css-modules/postcss-modules/blob/master/CHANGELOG.md. I think it's good to go but making sure you see this before merging.

@wardpeet
Copy link
Collaborator

wardpeet commented Dec 6, 2020

Thanks for your PR and sorry for the delay, it seems that #342 already fixes this.

@wardpeet wardpeet closed this Dec 6, 2020
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