-
Notifications
You must be signed in to change notification settings - Fork 4
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
2.x Prep and base #278
base: develop
Are you sure you want to change the base?
2.x Prep and base #278
Conversation
Holy cow this was hard to get committed. I had to ignore these tests for now. We must clean these up before this is ready: |
feat: Add no-platform starter as default init setup
Allow init command to run without parameters
feat: Switch from chalk to colorette for better compatibility
2.x cleanup commands
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.
@mikeethedude Should I be able to run:
emulsify component install blockquote --force
And have it, or any other component (https://github.com/emulsify-ds/emulsify-ui-kit/tree/main/src/components) I list, install successfully?
I tried installing the blockquote and I get no feedback
@joetower Only if that is defined here: https://github.com/emulsify-ds/emulsify-ui-kit/blob/main/system.emulsify.json |
@mikeethedude Okay, thank you! The thing is, in testing this, those three already exist, which, I suppose is why the |
Yeah you can try and remove that component or otherwise mess it up to see if that puts the right one in. |
@mikeethedude Thank you! I removed Note: it did add the |
@mikeethedude But, if I remove it again and run it with |
I believe that lines up with the expectations though I think it should be okay with a dependency existing. Maybe we should have it ask if you want to replace the dependency when it already exists? The red text is also probably a little strong there. This is good feedback, thank you! |
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.
@mikeethedude This tests as you've outlined. Super excited about this release and what will come next. 👍
2.x Clean experimental warnings
Better 2 x cache fix
Summary
This is where we are starting for 2.x Here are some of the thigs we need to accomplish in this branch.
Documentation update (required)
If this pull request requires a change to Emulsify documentation, those changes, updates, and/or new information must accompany this pull request.
How to review this pull request
emulsify init
Should take you through a wizard to installemulsify system install emulsify-ui-kit
emulsify component install text --force
Closing issues
Closes #