-
Notifications
You must be signed in to change notification settings - Fork 79
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: move packages/components into core #767
Conversation
🦋 Changeset detectedLatest commit: 2ae2ef0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Ignored Deployments
|
4700047
to
bf2079f
Compare
bf2079f
to
c9387d1
Compare
c9387d1
to
23a653e
Compare
23a653e
to
e3a386a
Compare
e3a386a
to
70aebe6
Compare
70aebe6
to
f64b123
Compare
f64b123
to
21246ab
Compare
Disabled Visual Regression tests to unblock this PR. |
Splitting PR into commits/multiple PRs. |
bbb8e2c
to
dcf3d4c
Compare
dcf3d4c
to
ff53081
Compare
ff53081
to
53341e2
Compare
pnpm-lock.yaml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you haven't already, I would make sure to rerun pnpm install
after you removed the old package.json
.
b4c3010
to
2ae2ef0
Compare
⚡️🏠 Lighthouse reportLighthouse ran against https://catalyst-latest-319x54960-bigcommerce-platform.vercel.app 🖥️ DesktopWe ran Lighthouse against the changes on a desktop and produced this report. Here's the summary:
📱 MobileWe ran Lighthouse against the changes on a mobile and produced this report. Here's the summary:
|
What/Why?
Move
packages/components
into corecomponents/ui
.package.json
andtailwind.config
into core.changeset
configuration.Testing
Locally.