-
Notifications
You must be signed in to change notification settings - Fork 350
chore: change packages to private #3154
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
Conversation
| /> | ||
|
|
||
| ## Use cases and examples | ||
| {/* ## Use cases and examples */} |
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.
Will revert in a different PR
|
📦 Bundle Size Analysis ✅ No bundle size changes detected. Unchanged Components
📊 Summary:
|
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Type
Enhancement, Other
Description
Mark internal packages as private in package.json
Remove circular dependency on @vibe/button from loader
Comment out loader-in-button story and related imports
Resolve circular dependency issue between components
Diagram Walkthrough
File Walkthrough
package.json
Mark config package as privatepackages/config/package.json
"private": trueflag to mark package as internalpackage.json
Mark shared package as privatepackages/shared/package.json
"private": trueflag to mark package as internalpackage.json
Mark storybook-config package as privatepackages/storybook-config/package.json
"private": trueflag to mark package as internalpackage.json
Remove button dependency from loadercomponents/loader/package.json
@vibe/buttonfrom devDependencies to resolve circulardependency
Loader.stories.tsx
Remove button-dependent story and importscomponents/loader/src/Loader/stories/Loader.stories.tsx
Loader.mdx
Comment out button-related documentationcomponents/loader/src/Loader/stories/Loader.mdx
component