Skip to content

Fix CI: replace internal registry URLs in yarn.lock#5274

Open
captbaritone wants to merge 1 commit into
facebook:mainfrom
captbaritone:fix-yarn-lock-registry
Open

Fix CI: replace internal registry URLs in yarn.lock#5274
captbaritone wants to merge 1 commit into
facebook:mainfrom
captbaritone:fix-yarn-lock-registry

Conversation

@captbaritone
Copy link
Copy Markdown
Contributor

Summary

  • The ShipIt sync from fbsource (D103172778, commit 3dd4655) updated hermes-parser and related packages to 0.36.0, but the yarn.lock was generated internally against registry.facebook.net
  • GitHub Actions runners can't authenticate to registry.facebook.net, causing all JS CI jobs (lint, tests, flow typecheck, build) to fail with 401 Unauthorized
  • This replaces 6 registry.facebook.net URLs with registry.yarnpkg.com in yarn.lock

Affected packages

  • babel-plugin-syntax-hermes-parser@0.36.0
  • flow-bin@0.312.1
  • hermes-eslint@0.36.0
  • hermes-estree@0.36.0
  • hermes-parser@0.36.0
  • prettier-plugin-hermes-parser@0.36.0

Test plan

  • CI JS Lint passes
  • CI JS Tests (Node 18.x, 20.x) pass
  • CI Flow Typecheck passes
  • CI Build Rust Compiler passes (uses yarn install on push to main)

The ShipIt sync from fbsource introduced yarn.lock entries resolved
against registry.facebook.net, which is inaccessible from GitHub
Actions runners. Replace with registry.yarnpkg.com.

Affected packages: babel-plugin-syntax-hermes-parser, flow-bin,
hermes-eslint, hermes-estree, hermes-parser,
prettier-plugin-hermes-parser.
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented May 6, 2026

@captbaritone has imported this pull request. If you are a Meta employee, you can view this in D104137865.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant