Skip to content

feat: update ICSS utils to current package#145

Merged
madyankin merged 1 commit intomadyankin:masterfrom
jquense:update-icss-utils
Nov 3, 2022
Merged

feat: update ICSS utils to current package#145
madyankin merged 1 commit intomadyankin:masterfrom
jquense:update-icss-utils

Conversation

@jquense
Copy link
Copy Markdown
Contributor

@jquense jquense commented Nov 1, 2022

👋 Hey there, sorry for the drive-by PR. I've updated the replaceValues function to use the most recent version of this util. The old package redirects to icss-utils now. This isn't just maintenance however, the updated replacer works better and will replace values in selectors, which doesn't work right now. This is relevant to me (the maintainer of astroturf) which depends on this behavior in webpack, and i am trying now to write a compatible Vite version

@jquense
Copy link
Copy Markdown
Contributor Author

jquense commented Nov 3, 2022

@madyankin Also if you'd like some help maintaining this, I'd be happy to. I know i'm a random person, but I have a lot of experience in the css-modules tooling. I also know how time consuming OSS is and appreciate you may not have a lot of time to look at this!

@madyankin madyankin merged commit dc5119e into madyankin:master Nov 3, 2022
@jquense jquense deleted the update-icss-utils branch November 3, 2022 17:27
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.

2 participants