-
Notifications
You must be signed in to change notification settings - Fork 79
feat: move Tailwind preset to package #11817
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
768dfc3
to
f803843
Compare
This PR has been automatically marked as stale because it has not had recent activity. Please close your PR if it is no longer relevant. Thank you for your contributions. |
Self-reminder: follow post-install steps from https://github.com/Esri/calcite-design-system/wiki/monorepo#adding-new-packages. |
Happy to hear any thoughts or suggestions on the README and CONTRIBUTING docs. 😄👂 |
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.
🌮
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.
Nice! Looking good, but the package will release as v1.1.0
instead of v1.0.0
.
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.
I think we will get errors publishing this package via the CI because it doesn't exist yet. The first time you publish a scoped package, the command needs --access public
.
I think the best option would be publishing 1.0.0
locally so we can specify the access. The changes I mentioned above would no longer be needed too.
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.
LGTM after one documentation suggestion!
Co-authored-by: Ben Elan <[email protected]>
**Related Issue:** #6602 ## Summary ✨📦✨ --------- Co-authored-by: Ben Elan <[email protected]>
Related Issue: #6602
Summary
✨📦✨