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

ci(rollup): inject styles with components in dist #4619

Closed
wants to merge 7 commits into from

Conversation

HeartSquared
Copy link
Contributor

Why

Inject styles with each component and prevent styles "going missing" in consumer repos.
As an extra benefit, this allows for the styles to be tree-shaken with its component.

Note that we still have @kaizen/components/dist/styles.css for CSS globals (variables, reset etc).

What

  • Inject styles per component

WIP

Agadoo (tree-shaking package) is falsely failing. Need to figure out how to tell it to ignore the generated style files.

Copy link

changeset-bot bot commented Apr 9, 2024

🦋 Changeset detected

Latest commit: a2becc0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@kaizen/components Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Base automatically changed from KZN-2208/fix-dist-cjs to main April 9, 2024 06:10
@HeartSquared HeartSquared force-pushed the heartsquared/dist-inject-styles branch from a2f1ad5 to 291a55d Compare April 9, 2024 06:27
@HeartSquared HeartSquared deleted the heartsquared/dist-inject-styles branch May 21, 2024 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant