From a19fd2413228b3d544536e60a80f092f311d2797 Mon Sep 17 00:00:00 2001 From: Rob Hogan Date: Sun, 9 Feb 2025 16:47:39 -0800 Subject: [PATCH] Scripts: remove checked-in `debugger;` statements Summary: These snuck in, presumably accidentally, via https://github.com/facebook/react-native/pull/49164. This interferes with running test suites with a debugger connected (e.g, when debugging Jest itself). (Aside: we should probably enable [`eslint/no-debugger`](https://eslint.org/docs/latest/rules/no-debugger) to catch these) Changelog: [Internal] Differential Revision: D69377992 --- .../workflow-scripts/__tests__/verifyPublishedPackage-test.js | 1 - .github/workflow-scripts/verifyPublishedPackage.js | 1 - 2 files changed, 2 deletions(-) diff --git a/.github/workflow-scripts/__tests__/verifyPublishedPackage-test.js b/.github/workflow-scripts/__tests__/verifyPublishedPackage-test.js index ca12bb9a646..95e22baa854 100644 --- a/.github/workflow-scripts/__tests__/verifyPublishedPackage-test.js +++ b/.github/workflow-scripts/__tests__/verifyPublishedPackage-test.js @@ -83,7 +83,6 @@ describe('#verifyPublishedPackage', () => { .mockImplementation(() => { throw new Error('Should not be called again!'); }); - debugger; await verifyPublishedPackage(REACT_NATIVE_PACKAGE, version, 'next'); expect(mockGetNpmPackageInfo).toHaveBeenCalledWith( diff --git a/.github/workflow-scripts/verifyPublishedPackage.js b/.github/workflow-scripts/verifyPublishedPackage.js index 7f84283ef26..825300d9b0e 100644 --- a/.github/workflow-scripts/verifyPublishedPackage.js +++ b/.github/workflow-scripts/verifyPublishedPackage.js @@ -18,7 +18,6 @@ async function verifyPublishedPackage( tag = null, retries = MAX_RETRIES, ) { - debugger; log(`🔍 Is ${packageName}@${version} on npm?`); let count = retries;