Skip to content

Conversation

@eps1lon
Copy link
Contributor

@eps1lon eps1lon commented Jul 31, 2024

Pinning it to its patch version contributes to dependency duplication in applications using basehub and @radix-ui/react-slot. @radix-ui/react-slot is not known for violating SemVer so pinning the version to its major is safe.

This ensures fixes are propagated faster throughout the ecosystem. In this case, @radix-ui/slot published a fix for React 19 compat that basehub users won't get until a new basehub release is out. It helps us in the React team in gathering feedback quicker since we don't need to ensure each dependent updates its dependencies even when their dependencies can ensure compatibility with a SemVer MINOR.

@changeset-bot
Copy link

changeset-bot bot commented Jul 31, 2024

🦋 Changeset detected

Latest commit: 249a647

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
basehub Patch
playground Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Jul 31, 2024

@eps1lon is attempting to deploy a commit to the BaseHub Team on Vercel.

A member of the Team first needs to authorize it.

@eps1lon eps1lon marked this pull request as ready for review July 31, 2024 13:57
Pinning it to its patch version contributes to dependency duplication in applications using `basehub` and `@radix-ui/react-slot`.
The package is not known for violating SemVer so pinning the version to its major is safe.

This ensures fixes are propagated faster throughout the ecosystem. In this case, `@radix-ui/slot` published a fix for React 19 compat that `basehub` users won't get until a new `basehub` release is out. It helps us in the React team in gathering feedback quicker since we don't need to ensure each dependent updates its dependencies even when their dependencies can ensure compatibility with a SemVer MINOR.
@eps1lon eps1lon mentioned this pull request Jul 31, 2024
22 tasks
@julianbenegas julianbenegas self-requested a review August 2, 2024 23:49
Copy link
Member

@julianbenegas julianbenegas left a comment

Choose a reason for hiding this comment

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

Thank you @eps1lon 🧡

@julianbenegas julianbenegas merged commit 176c0a7 into basehub-ai:main Aug 2, 2024
@eps1lon eps1lon deleted the radix-pin-major branch August 5, 2024 19:03
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.

2 participants