Skip to content

Consolidate package checks into one Jest test#57509

Closed
huntie wants to merge 1 commit into
react:mainfrom
huntie:export-D111471313
Closed

Consolidate package checks into one Jest test#57509
huntie wants to merge 1 commit into
react:mainfrom
huntie:export-D111471313

Conversation

@huntie

@huntie huntie commented Jul 10, 2026

Copy link
Copy Markdown
Member

Summary:
Simplify/unify various mechanisms for the monorepo's package invariants.

These checks were previously scattered:

  • private/monorepo-tests package (manifest field checks)
  • .github/workflow-scripts/lint_files.sh (.npmignore ban)
  • react-native/eslint-plugin-monorepo (manifest field checks — duplicated)

This folds everything into scripts/monorepo-tests/__tests__/check-packages-test.js. A plain Jest test is the most extensible home for future checks.

Changelog: [Internal]

Differential Revision: D111471313

Summary:
Simplify/unify various mechanisms for the monorepo's package invariants.

These checks were previously scattered:
- `private/monorepo-tests` package (manifest field checks)
- `.github/workflow-scripts/lint_files.sh` (`.npmignore` ban)
- `react-native/eslint-plugin-monorepo` (manifest field checks — duplicated)

This folds everything into `scripts/monorepo-tests/__tests__/check-packages-test.js`. A plain Jest test is the most extensible home for future checks.

Changelog: [Internal]

Differential Revision: D111471313
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 10, 2026

@cortinico cortinico left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

@meta-codesync meta-codesync Bot closed this in 0ed6c56 Jul 10, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Jul 10, 2026
@meta-codesync

meta-codesync Bot commented Jul 10, 2026

Copy link
Copy Markdown

This pull request has been merged in 0ed6c56.

@huntie huntie deleted the export-D111471313 branch July 10, 2026 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants