Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Rename npm package to @primer/react #1676

Merged
merged 7 commits into from
Dec 8, 2021
Merged

Rename npm package to @primer/react #1676

merged 7 commits into from
Dec 8, 2021

Conversation

colebemis
Copy link
Contributor

@colebemis colebemis commented Dec 1, 2021 •
edited
Loading

Part of https://github.com/github/primer/issues/241

Background

Primer React used to be called Primer Components. When we introduced Primer ViewComponents, we renamed Primer Components to Primer React to avoid confusion. However, we did not update the name consistently across all platforms.

Summary

  • Renames npm package from @primer/components to @primer/react
  • Updates all references to @primer/components in this repo

Rollout plan

  • Release final update (v33.0.0) to @primer/components package (Release Tracking #1667)
  • Merge this PR (This will trigger changesets to open a release tracking PR for @primer/react@34.0.0)
  • Release @primer/react@34.0.0
  • Deprecate @primer/components package with npm deprecate
  • Update references to @primer/components in eslint-plugin-primer-react
  • Update references to @primer/components in Doctocat
  • Announce package name change in #primer-react Slack channel

@changeset-bot
Copy link

changeset-bot bot commented Dec 1, 2021 •
edited
Loading

🦋 Changeset detected

Latest commit: 86d2a05

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

This PR includes changesets to release 1 package
Name Type
@primer/react Major

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

@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2021 •
edited
Loading

size-limit report 📦

Path Size
dist/browser.esm.js 57.25 KB (0%)
dist/browser.umd.js 57.6 KB (-0.01% 🔽)

@colebemis colebemis marked this pull request as ready for review December 1, 2021 19:03
@colebemis colebemis requested review from a team and mperrotti December 1, 2021 19:03
@colebemis colebemis added the react label Dec 1, 2021
jfuchs
jfuchs approved these changes Dec 1, 2021
View reviewed changes
Copy link
Contributor

@jfuchs jfuchs left a comment

Choose a reason for hiding this comment

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

cats-smoke

docs/gatsby-config.js Show resolved Hide resolved
@colebemis colebemis merged commit 1195336 into main Dec 8, 2021
@colebemis colebemis deleted the rename-npm-package branch December 8, 2021 19:06
@github-actions github-actions bot mentioned this pull request Dec 8, 2021
Merged
pksjce pushed a commit that referenced this pull request Dec 20, 2021
@colebemis @pksjce
Rename npm package to @primer/react (#1676)
d2abf97 
* Rename npm package to `@primer/react`

* Create rude-vans-guess.md

* Update rude-vans-guess.md

* Update docs

* Update package description

* Fix TextInput docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jfuchs jfuchs jfuchs approved these changes

@mperrotti mperrotti Awaiting requested review from mperrotti

Assignees
No one assigned
Labels
react
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@colebemis @jfuchs