-
Notifications
You must be signed in to change notification settings - Fork 120
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
Update the primer/primitives dependency to be 9.x || 10.x and move it to peer #3179
Conversation
… to peer Update the `@primer/primitives` dependency to `9.x || 10.x` and move it to peer dependencies. * **demo/package.json** - Update the `@primer/primitives` version to `9.x || 10.x`. - Move the `@primer/primitives` entry from `dependencies` to `peerDependencies`. * **package.json** - Update the `@primer/primitives` version to `9.x || 10.x`. - Move the `@primer/primitives` entry from `dependencies` to `peerDependencies`. * **package-lock.json** - Update the `@primer/primitives` version to `9.x || 10.x`. - Add `peerDependencies` section with `@primer/primitives` entry. * **demo/package-lock.json** - Update the `@primer/primitives` version to `9.x || 10.x`. - Add `peerDependencies` section with `@primer/primitives` entry. --- For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/primer/view_components?shareId=XXXX-XXXX-XXXX-XXXX).
🦋 Changeset detectedLatest commit: 9bc95c8 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
* Remove `@primer/primitives` entry from `dependencies` section
demo/package.json
Outdated
"@rails/actioncable": "^7.2.100", | ||
"@rails/ujs": "^7.1.400", | ||
"turbolinks": "^5.2.0" | ||
}, | ||
"peerDependencies": { | ||
"@primer/primitives": "9.x || 10.x" | ||
}, |
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.
This change in the demo isn't necessary since we don't publish this package, it only effects what's being used in the lookbook
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.
Fixed this
Closes https://github.com/github/primer/issues/4326
Update the
@primer/primitives
dependency to9.x || 10.x
and move it to peer dependencies.package.json
@primer/primitives
version to9.x || 10.x
.@primer/primitives
entry fromdependencies
topeerDependencies
.package-lock.json
@primer/primitives
version to9.x || 10.x
.peerDependencies
section with@primer/primitives
entry.For more details, open the Copilot Workspace session.