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.

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

Discussion: Removing "react-native" from repo names and moving packages to @react-native-community #33

Open
xzilja opened this issue Jun 10, 2019 · 14 comments
Labels
discussion Let's talk about stuff

Comments

@xzilja
Copy link

xzilja commented Jun 10, 2019

Proposal

I want to propose 2 changes, that (in my opinion) could further improve "consistency" feeling around react-native-community org

  1. Strip react-native from package names under react-native-community since organisation name already implies it.
  2. Bring all packages under @react-native-community/ namespace in package registry

Discussion

  1. How would maintainers feel about this?
  2. Difficulties that can be caused by renaming repos in GitHub and possible solutions to solve them?
  3. Difficulties that can be caused by renaming / moving packages into @react-native-community/ namespace and possible solutions to solve them?
  4. And in general is this something worth doing due to effort it requires (do all maintainers need to do this or can one person make the change across the org?)

Present status

There are several examples that show case this proposal within the community already: cli, apple-sign-in, art etc..

I also mentioned this in react-native-svg repository via software-mansion/react-native-svg#1021

@xzilja xzilja changed the title Discussion: strip "react-native" from library names and move them to [@react-native-community/lib]] Discussion: strip "react-native" from repo names and move packages to @react-native-community Jun 10, 2019
@xzilja xzilja changed the title Discussion: strip "react-native" from repo names and move packages to @react-native-community Discussion: Removing "react-native" from repo names and moving packages to @react-native-community Jun 10, 2019
@kelset
Copy link
Contributor

kelset commented Jun 10, 2019

Uhm I'm fairly sure there was already a conversation on this somewhere... I can't find it right now.

@kelset kelset added the discussion Let's talk about stuff label Jun 10, 2019
@matt-oakes
Copy link
Collaborator

Renaming Github repos is a good idea and shouldn't come with many downsides. Github automatically sets up redirects for the old names, so it should be fairly straightforward.

Releasing everything under the @react-native-community NPM scope would also be great, however, it might be a pain for users to need to migrate over and possibly leave some users who are unaware of the change "stranded" on older versions. Possible solutions:

Maybe it's worth talking to the maintainers of a library which has made a change to NPM package names about their experience with it? Babel comes to mind.

@pvinis
Copy link
Contributor

pvinis commented Jun 11, 2019

Also, what do you mean by "bring all packages under RNC"?

@xzilja
Copy link
Author

xzilja commented Jun 11, 2019

@pvins @react-native-community org over at npm

@thymikee
Copy link
Contributor

Fully supportive, we already renamed a bunch, like CLI or ART.

@sibelius
Copy link
Contributor

@sbeca wants to move https://github.com/react-native-community/react-native-audio-toolkit under @react-native-community scoped package

how can he do that?

@sbeca
Copy link
Contributor

sbeca commented Jun 15, 2019

Yeah, react-native-audio-toolkit was transferred from Futurice's Github account to the react-native-community Github account a few months ago but is still under Futurice's npm account. What is the process for transferring the npm package to the react-native-community npm account?

@xzilja
Copy link
Author

xzilja commented Jun 16, 2019

@thymikee when you have a minute, could you please outline steps that you guys took in order to make this happen? Once we have solid "miniguide" I could open proposal issues in each repo mentioning those (hopefully this won't be too spammy, but rather ensure all maintainers are aware?).

I guess we need to know how to: rename repo to remove 'react-native' and move npm package under rnc org with least resistance to maintainers and end users.

@thymikee
Copy link
Contributor

thymikee commented Jun 16, 2019

  1. Make sure the package is published under RNC scope to npm
  2. Go to the settings
  3. Change the repo name to be exactly the same as npm package
  4. Update references to the repo address if any (e.g links in error messages)

That's it, GH takes care of everything else for us.

If the package is not published as scoped, then it needs a discussion, but we should strive to have it sooner or later.

@matt-oakes
Copy link
Collaborator

If we're going to rename the NPM packages, we should have a plan to make it clear to existing users what they need to do to migrate. Possible options are:

  • Continue to publish to both packages. This would be easier to do of using an automated tool.
  • Mark the old package as deprecated and add information about the new name.
  • Make sure the README has information about how to migrate, including how to redo the linking.

I recently went through this as a library user with RNC Blur View. I only noticed that the package name as changed after going to the GitHub page. If I hadn't have done that I would be "stuck" on the old version.

@sbeca
Copy link
Contributor

sbeca commented Jun 22, 2019

So, for react-native-audio-toolkit I should do the following:

  1. Publish v2.0.0 to NPM as a new NPM package: @react-native-community/audio-toolkit
  2. Publish v1.0.7 to NPM under the old NPM package, which will just be the current version with a deprecation warning pointing to the new version.

I am not part of the react-native-community NPM organisation yet so who can help me with step 1? My NPM username is scott.beca if someone is looking to invite me to the org.

@MateusAndrade
Copy link
Contributor

I am not part of the react-native-community NPM organisation yet so who can help me with step 1? My NPM username is scott.beca if someone is looking to invite me to the org.

I think @kelset can give you a hand with that 😃

@kelset
Copy link
Contributor

kelset commented Jun 24, 2019

👋 @sbeca.
Actually I don't have the NPM level of auth needed, but I'll ping who has.

In the meantime, there are a few other house cleaning things that I'd need your help with (for example, in the maintainers https://github.com/react-native-community/.github/blob/master/MAINTAINERS.md you are not listed 😅).

If you are on twitter, could you send me a dm there so that I can guide you through a few of these things?

@sbeca
Copy link
Contributor

sbeca commented Jun 24, 2019

I'll send you a DM now @kelset

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Let's talk about stuff
Projects
None yet
Development

No branches or pull requests

8 participants