chore: remove home field from npmrc

[HomyeeKing]

npm warn Unknown user config "home". This will stop working in the next major version of npm.

the home config was decrepated, so it'd better remove it from here as well.

[changeset-bot]

🦋 Changeset detected

Latest commit: 0424253

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

This PR includes changesets to release 1 package

Name	Type
nrm	Minor

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

[HomyeeKing]

@iosh cc

[iosh]

The test failed. I'll check it later

[HomyeeKing]

@iosh it's the nrm home command failed, should we remove this command as well?

[iosh]

@iosh it's the nrm home command failed, should we remove this command as well?

Yes, you're right, The nrm home command we should remove it.

[HomyeeKing]

@iosh Maybe we could remain the home command and move the home field into ~/.nrmrc, so we can get rid of the warn message, what do you think?

[iosh]

@iosh Maybe we could remain the home command and move the home field into ~/.nrmrc, so we can get rid of the warn message, what do you think?

I think we can either remove the command now or add a warning first and remove it later.

[HomyeeKing]

@iosh
The intention of this PR is to get rid of the warning from npm, so I simply remove the home config before I read the code.

Now I realise the nrm home command is one of the feature of the nrm and I suggest to keep it, just put the NON_NPM_CONFIG in the nvmrc will be fine.

The warning disappear and no harm to the feature.

[iosh]

I think we can stop write home field to .npmrc

export async function onUse(name: string) {
  const registries = await getRegistries();
  let alias = name;

  // if alias is undefined, select the registry alias from list
  if (alias === undefined) {
    alias = await select<string>({
      message: 'Please select the registry you want to use',
      choices: Object.keys(registries),
      pageSize: 10,
    });
  }

  if (await isRegistryNotFound(alias)) {
    return;
  }

  const registry = registries[alias];
  const npmrc = await readFile(NPMRC);
  await writeFile(NPMRC, Object.assign(npmrc, registry));

  printSuccess(`The registry has been changed to '${alias}'.`);
}

export async function onUse(name: string) {
  // ...
  // Don't write the`home` to .npmrc
  const {home:_, ...registry} = registries[alias];
  const npmrc = await readFile(NPMRC);
  await writeFile(NPMRC, Object.assign(npmrc, registry));

  printSuccess(`The registry has been changed to '${alias}'.`);
}

[iosh]

@HomyeeKing Thanks, Could you please add a changeset for this PR? Please run npx changeset, choose minor, and use the following summary:: fix: Remove invalid home field from .npmrc. Thanks @HomyeeKing

[HomyeeKing]

@iosh Done, thanks :)