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

feat: init remove command #15

Merged
merged 23 commits into from
Apr 14, 2024
Merged

feat: init remove command #15

merged 23 commits into from
Apr 14, 2024

Conversation

winchesHe
Copy link
Member

@winchesHe winchesHe commented Apr 7, 2024

📝 Description

1、Main

Remove the selected component

If there are no any ohters NextUI components then remove @nextui-org/system @nextui-org/theme, remove tailwind, .npmrc (only pnpm) too

CleanShot.2024-04-11.at.11.53.54.mp4

2、Remove all

CleanShot.2024-04-11.at.15.15.20.mp4

3、Illegal Components check

image

4、Warn when there are no NextUI Components

Uploading image.png…

✅ Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (improve a current implementation without adding a new feature or fixing a bug)
  • Improvement (non-breaking change which improves an existing feature)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

@winchesHe winchesHe marked this pull request as ready for review April 9, 2024 16:11
@winchesHe winchesHe changed the title WIP: remove command feat: init remove command Apr 9, 2024
@winchesHe
Copy link
Member Author

WIP: Testing

@jrgarciadev
Copy link
Member

@winchesHe could you please fix the conflicts?

@jrgarciadev jrgarciadev merged commit 6983a9b into main Apr 14, 2024
2 of 3 checks passed
@jrgarciadev jrgarciadev deleted the feature-remove branch April 14, 2024 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants