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

fix(rollup-conf): styles dir location #155

Merged
merged 1 commit into from May 16, 2020
Merged

fix(rollup-conf): styles dir location #155

merged 1 commit into from May 16, 2020

Conversation

Thomaash
Copy link
Member

Due to recent breaking changes in Rollup PostCSS plugin that I completely missed the styles dir has moved from the root to the bundle files and is now 4 times in the packages (4 copies of the same thing). This is not how it is documented and was never announced to the public as a breaking change. This commit reverses it to the way it used to be prior to the update. It also officially drops support for Rollup PostCSS plugin (dev dep in our projects) version 2 since the way paths are interpreted is different from the up to date version 3.

Due to recent breaking changes in Rollup PostCSS plugin that I
completely missed the styles dir has moved from the root to the bundle
files and is now 4 times in the packages (4 copies of the same thing).
This is not how it is documented and was never announced to the public
as a breaking change. This commit reverses it to the way it used to be
prior to the update. It also officially drops support for Rollup PostCSS
plugin (dev dep in our projects) version 2 since the way paths are
interpreted is different from the up to date version 3.
@Thomaash Thomaash requested a review from a team May 15, 2020 22:07
@yotamberk yotamberk merged commit 3c7866e into master May 16, 2020
@yotamberk yotamberk deleted the fix-styles-dir branch May 16, 2020 05:43
@vis-bot
Copy link
Collaborator

vis-bot commented May 16, 2020

🎉 This PR is included in version 2.2.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

3 participants