Skip to content

Commit eec2642

Browse files
javachemeta-codesync[bot]
authored andcommitted
Move remaining js1 build-* commands under the build namespace (#57348)
Summary: Pull Request resolved: #57348 Follow-up to the `js1 build-cxx-api` rename. Move `build-geo-screenshot-tests`, `build-js-api`, and `build-twui-screenshot-tests` under the `build` group so they sit next to their siblings (`js1 build assets`, `js1 build turbomodule`, etc.) instead of being top-level hyphenated outliers. Each old top-level command stays as a hidden alias that prints a deprecation notice and delegates to the new module, so existing muscle memory and any out-of-tree scripts keep working. Also updates the two places that emit the old command names into user-visible output: the TWUI template comment headers, and the JS API snapshot failure message. Changelog: [Internal] Reviewed By: zeyap Differential Revision: D109844649 fbshipit-source-id: 00623fa8b4d724e16a2dc62d196182c32a9fed64
1 parent 54b88d9 commit eec2642

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

scripts/js-api/build-types/buildApiSnapshot.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ async function validateSnapshots(
148148
// $FlowFixMe[prop-missing]
149149
// $FlowFixMe[incompatible-type]
150150
const diffResult = diff(prevSnapshot, newSnapshot, options);
151-
const rerunCommand = isGitRepo() ? 'yarn build-types' : 'js1 build-js-api';
151+
const rerunCommand = isGitRepo() ? 'yarn build-types' : 'js1 build js-api';
152152
console.error(
153153
`${styleText(['bold', 'inverse', 'red'], ' FAIL ')} ReactNativeApi.d.ts has changed. Please re-run '${rerunCommand}' and commit the updated snapshot.\n`,
154154
);

0 commit comments

Comments
 (0)