Skip to content

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

Merged
merged 18 commits into from
Apr 9, 2025

Conversation

jcfranco
Copy link
Member

@jcfranco jcfranco commented Mar 28, 2025

Related Issue: #6602

Summary

✨📦✨

@jcfranco jcfranco force-pushed the jcfranco/6602-publish-tailwind-config branch from 768dfc3 to f803843 Compare March 28, 2025 01:35
@github-actions github-actions bot added the chore Issues with changes that don't modify src or test files. label Mar 28, 2025
@jcfranco jcfranco changed the title chore: release main (#10457) feat: move Tailwind preset to package Mar 28, 2025
@github-actions github-actions bot added the enhancement Issues tied to a new feature or request. label Mar 28, 2025
@jcfranco jcfranco added the pr ready for visual snapshots Adding this label will run visual snapshot testing. label Mar 28, 2025
Copy link
Contributor

github-actions bot commented Apr 5, 2025

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.

@github-actions github-actions bot added the Stale Issues or pull requests that have not had recent activity. label Apr 5, 2025
@jcfranco
Copy link
Member Author

jcfranco commented Apr 7, 2025

Self-reminder: follow post-install steps from https://github.com/Esri/calcite-design-system/wiki/monorepo#adding-new-packages.

@jcfranco jcfranco marked this pull request as ready for review April 8, 2025 07:20
@jcfranco jcfranco added pr ready for visual snapshots Adding this label will run visual snapshot testing. and removed pr ready for visual snapshots Adding this label will run visual snapshot testing. Stale Issues or pull requests that have not had recent activity. labels Apr 8, 2025
@jcfranco
Copy link
Member Author

jcfranco commented Apr 8, 2025

Happy to hear any thoughts or suggestions on the README and CONTRIBUTING docs. 😄👂

Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🌮

@jcfranco jcfranco added pr ready for visual snapshots Adding this label will run visual snapshot testing. and removed pr ready for visual snapshots Adding this label will run visual snapshot testing. labels Apr 8, 2025
Copy link
Member

@benelan benelan left a 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.

Copy link
Member

@benelan benelan left a 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.

@jcfranco
Copy link
Member Author

jcfranco commented Apr 9, 2025

@benelan Published as 0.1.0! 🎉

@jcfranco jcfranco added skip visual snapshots Pull requests that do not need visual regression testing. and removed pr ready for visual snapshots Adding this label will run visual snapshot testing. labels Apr 9, 2025
Copy link
Member

@benelan benelan left a 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!

@jcfranco jcfranco merged commit ce9595c into dev Apr 9, 2025
10 checks passed
@jcfranco jcfranco deleted the jcfranco/6602-publish-tailwind-config branch April 9, 2025 22:21
@github-actions github-actions bot added this to the 2025-04-29 - Apr Milestone milestone Apr 9, 2025
benelan added a commit that referenced this pull request May 14, 2025
**Related Issue:** #6602 

## Summary

✨📦✨

---------

Co-authored-by: Ben Elan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issues with changes that don't modify src or test files. enhancement Issues tied to a new feature or request. skip visual snapshots Pull requests that do not need visual regression testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants