Page MenuHomePhabricator

[yarn] Lift `.npmrc` up from `keyserver` to root of repo
ClosedPublic

Authored by atul on May 2 2022, 10:59 AM.
Tags
None
Referenced Files
F3179724: D3888.id12156.diff
Fri, Nov 8, 3:30 AM
F3178908: D3888.id12139.diff
Fri, Nov 8, 2:37 AM
F3178117: D3888.diff
Fri, Nov 8, 1:02 AM
Unknown Object (File)
Thu, Oct 17, 7:48 PM
Unknown Object (File)
Thu, Oct 17, 7:48 PM
Unknown Object (File)
Thu, Oct 17, 7:48 PM
Unknown Object (File)
Thu, Oct 17, 7:46 PM
Unknown Object (File)
Wed, Oct 16, 8:42 AM

Details

Summary

Addresses the issue with yarn upgrade-interactive documented by @benschac in ENG-1063: https://linear.app/comm/issue/ENG-1063/yarn-upgrade-interactive-functionality-errors

Basically, found that yarn upgrade-interactive failed in every yarn workspace except for keyserver. Was wondering what might be different, and remembered that we had to add an entry to keyserver/.npmrc in order for things to work there (https://phabricator.ashoat.com/D2966).

Tried moving .npmrc to the root of the repo and yarn upgrade-interactive worked fine without errors. I don't have much context on why we needed the .npmrc in the first place, but remember there was some weirdness with matrix/gitlab/etc... (would be good to get @marcinwasowicz's perspective here).

Test Plan
  1. Run yarn upgrade-interactive in the root of the repo.
  2. See if the yarn upgrade-interactive TUI shows up without displaying the "Can't find blah..." error message.

Did not actually try upgrading any packages with this interface. Will end up testing that implicitly the next time I upgrade a package (usually just modify package.json, but can use this approach just to make sure it works)

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul requested review of this revision.May 2 2022, 11:05 AM
This revision is now accepted and ready to land.May 2 2022, 11:50 AM

Removing @marcinwasowicz as blocking reviewer since I think this is good to go and he's off for a few days, but would still be good for him to take a look.

I tried it.

It's working, but at a snails speed. It could just be my internet connection though.

It's working, but at a snails speed. It could just be my internet connection though.

It's slow for me as well on a pretty fast network:

44a6.png (830×1 px, 102 KB)

Guessing the dependency graph is just big or something