Skip to content

feat(carbon-react): add public style entrypoints#11009

Merged
kodiakhq[bot] merged 11 commits into
carbon-design-system:mainfrom
joshblack:feat/add-style-entrypoints-and-tests
Mar 17, 2022
Merged

feat(carbon-react): add public style entrypoints#11009
kodiakhq[bot] merged 11 commits into
carbon-design-system:mainfrom
joshblack:feat/add-style-entrypoints-and-tests

Conversation

@joshblack

Copy link
Copy Markdown
Contributor

This PR updates our @carbon/react package to include all public sass files. It also includes tests for these files and a common entrypoint for finding common files from the styles package.

Changelog

New

  • Tests for sass files in @carbon/react
  • Common entrypoint for public sass files in @carbon/styles
  • New files for public sass files

Changed

  • Build task for @carbon/react styles

Removed

Testing / Reviewing

  • Verify tests pass as expected
  • Verify all public files are included in the packages/styles/files.js file

@joshblack joshblack requested a review from tay1orjones March 17, 2022 15:42
@joshblack joshblack requested a review from a team as a code owner March 17, 2022 15:42
@joshblack joshblack requested a review from abbeyhrt March 17, 2022 15:42
@netlify

netlify Bot commented Mar 17, 2022

Copy link
Copy Markdown

✔️ Deploy Preview for carbon-react-next ready!

🔨 Explore the source changes: 967c5b7

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-react-next/deploys/62337fe46736a60008a3d93c

😎 Browse the preview: https://deploy-preview-11009--carbon-react-next.netlify.app

@netlify

netlify Bot commented Mar 17, 2022

Copy link
Copy Markdown

✔️ Deploy Preview for carbon-elements ready!

🔨 Explore the source changes: 967c5b7

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-elements/deploys/62337fe4ee100c00082ac3b4

😎 Browse the preview: https://deploy-preview-11009--carbon-elements.netlify.app

@netlify

netlify Bot commented Mar 17, 2022

Copy link
Copy Markdown

✔️ Deploy Preview for carbon-components-react ready!
Built without sensitive environment variables

🔨 Explore the source changes: 967c5b7

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-components-react/deploys/62337fe4c8edd90008716d15

😎 Browse the preview: https://deploy-preview-11009--carbon-components-react.netlify.app

Comment on lines +26 to +28
"peerDependencies": {
"sass": "^1.32.12"
},

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on #10600 we had said we'd be requiring sass@>=1.33.0

"@carbon/test-utils": "^10.21.0",
"css": "^3.0.0"
"css": "^3.0.0",
"sass": "^1.45.0"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same thing here on the sass version

@tay1orjones tay1orjones left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small blip with the sass version, but I can issue a follow-up PR to close out #10600 separately.

@kodiakhq kodiakhq Bot merged commit 8d07337 into carbon-design-system:main Mar 17, 2022
tay1orjones pushed a commit to tay1orjones/carbon that referenced this pull request Mar 18, 2022
…m#11009)

* chore(styles): add list for public files

* chore(carbon-react): update build-styles task

* feat(carbon-react): add in all public sass files

* chore(styles): update files export to include full import path

* chore(carbon-react): fix build-styles task

* chore(styles): update file output

* fix(carbon-react): update root index.scss

* chore(styles): add relative path for tests

* test(carbon-react): add scss tests

* chore(project): update sass deps

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
@tay1orjones tay1orjones mentioned this pull request Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants