Skip to content

Commit bb0d069

Browse files
committed
[build2 -> build] Local scripts
Update all our local scripts to use `build` instead of `build2`. There are still downstream scripts that depend on `build2`, though, so we can't remove it yet.
1 parent 162bb8c commit bb0d069

File tree

14 files changed

+19
-23
lines changed

14 files changed

+19
-23
lines changed

fixtures/dom/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
},
2121
"scripts": {
2222
"start": "react-scripts start",
23-
"prestart": "cp ../../build2/oss-stable/scheduler/umd/scheduler-unstable_mock.development.js ../../build2/oss-stable/scheduler/umd/scheduler-unstable_mock.production.min.js ../../build2/oss-stable/react/umd/react.development.js ../../build2/oss-stable/react-dom/umd/react-dom.development.js ../../build2/oss-stable/react/umd/react.production.min.js ../../build2/oss-stable/react-dom/umd/react-dom.production.min.js ../../build2/oss-stable/react-dom/umd/react-dom-server.browser.development.js ../../build2/oss-stable/react-dom/umd/react-dom-server.browser.production.min.js ../../build2/oss-stable/react-dom/umd/react-dom-test-utils.development.js ../../build2/oss-stable/react-dom/umd/react-dom-test-utils.production.min.js public/ && cp -a ../../build2/oss-stable/. node_modules",
23+
"prestart": "cp ../../build/oss-stable/scheduler/umd/scheduler-unstable_mock.development.js ../../build/oss-stable/scheduler/umd/scheduler-unstable_mock.production.min.js ../../build/oss-stable/react/umd/react.development.js ../../build/oss-stable/react-dom/umd/react-dom.development.js ../../build/oss-stable/react/umd/react.production.min.js ../../build/oss-stable/react-dom/umd/react-dom.production.min.js ../../build/oss-stable/react-dom/umd/react-dom-server.browser.development.js ../../build/oss-stable/react-dom/umd/react-dom-server.browser.production.min.js ../../build/oss-stable/react-dom/umd/react-dom-test-utils.development.js ../../build/oss-stable/react-dom/umd/react-dom-test-utils.production.min.js public/ && cp -a ../../build/oss-stable/. node_modules",
2424
"build": "react-scripts build && cp build/index.html build/200.html",
2525
"test": "react-scripts test --env=jsdom",
2626
"eject": "react-scripts eject"

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@
108108
"scripts": {
109109
"build": "node ./scripts/rollup/build.js",
110110
"build-combined": "node ./scripts/rollup/build-all-release-channels.js",
111-
"build-for-devtools": "cross-env RELEASE_CHANNEL=experimental yarn build react/index,react/jsx,react-dom,react-is,react-debug-tools,scheduler,react-test-renderer,react-refresh --type=NODE && rm -rf build2 && mkdir build2 && cp -r ./build/node_modules build2/oss-experimental/",
111+
"build-for-devtools": "cross-env RELEASE_CHANNEL=experimental yarn build react/index,react/jsx,react-dom,react-is,react-debug-tools,scheduler,react-test-renderer,react-refresh --type=NODE && rm -rf build && mkdir build && cp -r ./build/node_modules build/oss-experimental/",
112112
"build-for-devtools-dev": "yarn build-for-devtools --type=NODE_DEV",
113113
"build-for-devtools-prod": "yarn build-for-devtools --type=NODE_PROD",
114114
"linc": "node ./scripts/tasks/linc.js",

packages/react-devtools-core/webpack.backend.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ const builtModulesDir = resolve(
2222
__dirname,
2323
'..',
2424
'..',
25-
'build2',
25+
'build',
2626
'oss-experimental',
2727
);
2828

packages/react-devtools-core/webpack.standalone.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ const builtModulesDir = resolve(
2222
__dirname,
2323
'..',
2424
'..',
25-
'build2',
25+
'build',
2626
'oss-experimental',
2727
);
2828

packages/react-devtools-extensions/build.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ const ensureLocalBuild = async () => {
2727
__dirname,
2828
'..',
2929
'..',
30-
'build2',
30+
'build',
3131
'oss-experimental',
3232
);
3333

packages/react-devtools-extensions/webpack.backend.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ const builtModulesDir = resolve(
2424
__dirname,
2525
'..',
2626
'..',
27-
'build2',
27+
'build',
2828
'oss-experimental',
2929
);
3030

packages/react-devtools-extensions/webpack.config.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ const builtModulesDir = resolve(
2424
__dirname,
2525
'..',
2626
'..',
27-
'build2',
27+
'build',
2828
'oss-experimental',
2929
);
3030

packages/react-devtools-shell/webpack.config.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ const builtModulesDir = resolve(
2828
__dirname,
2929
'..',
3030
'..',
31-
'build2',
31+
'build',
3232
'oss-experimental',
3333
);
3434

scripts/circleci/check_minified_errors.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
# Ensure errors are minified in production
44

5-
OUT=$(git --no-pager grep -n --untracked --no-exclude-standard 'FIXME (minify-errors-in-prod)' -- './build2/*')
5+
OUT=$(git --no-pager grep -n --untracked --no-exclude-standard 'FIXME (minify-errors-in-prod)' -- './build/*')
66

77
if [ "$OUT" != "" ]; then
88
echo "$OUT";

scripts/jest/config.build-devtools.js

+3-5
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,11 @@ moduleNameMapper['react-devtools-feature-flags'] =
3535
// Map packages to bundles
3636
packages.forEach(name => {
3737
// Root entry point
38-
moduleNameMapper[
39-
`^${name}$`
40-
] = `<rootDir>/build2/${NODE_MODULES_DIR}/${name}`;
38+
moduleNameMapper[`^${name}$`] = `<rootDir>/build/${NODE_MODULES_DIR}/${name}`;
4139
// Named entry points
4240
moduleNameMapper[
4341
`^${name}\/([^\/]+)$`
44-
] = `<rootDir>/build2/${NODE_MODULES_DIR}/${name}/$1`;
42+
] = `<rootDir>/build/${NODE_MODULES_DIR}/${name}/$1`;
4543
});
4644

4745
// Allow tests to import shared code (e.g. feature flags, getStackByFiberInDevAndProd)
@@ -57,7 +55,7 @@ module.exports = Object.assign({}, baseConfig, {
5755
// Exclude the build output from transforms
5856
transformIgnorePatterns: [
5957
'/node_modules/',
60-
'<rootDir>/build2/',
58+
'<rootDir>/build/',
6159
'/__compiled__/',
6260
'/__untransformed__/',
6361
],

scripts/jest/config.build.js

+3-5
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,11 @@ moduleNameMapper[
3838
// Map packages to bundles
3939
packages.forEach(name => {
4040
// Root entry point
41-
moduleNameMapper[
42-
`^${name}$`
43-
] = `<rootDir>/build2/${NODE_MODULES_DIR}/${name}`;
41+
moduleNameMapper[`^${name}$`] = `<rootDir>/build/${NODE_MODULES_DIR}/${name}`;
4442
// Named entry points
4543
moduleNameMapper[
4644
`^${name}\/([^\/]+)$`
47-
] = `<rootDir>/build2/${NODE_MODULES_DIR}/${name}/$1`;
45+
] = `<rootDir>/build/${NODE_MODULES_DIR}/${name}/$1`;
4846
});
4947

5048
module.exports = Object.assign({}, baseConfig, {
@@ -58,7 +56,7 @@ module.exports = Object.assign({}, baseConfig, {
5856
// Don't run bundle tests on -test.internal.* files
5957
testPathIgnorePatterns: ['/node_modules/', '-test.internal.js$'],
6058
// Exclude the build output from transforms
61-
transformIgnorePatterns: ['/node_modules/', '<rootDir>/build2/'],
59+
transformIgnorePatterns: ['/node_modules/', '<rootDir>/build/'],
6260
setupFiles: [
6361
...baseConfig.setupFiles,
6462
require.resolve('./setupTests.build.js'),

scripts/jest/jest-cli.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,7 @@ function validateOptions() {
219219

220220
if (argv.build) {
221221
// TODO: We could build this if it hasn't been built yet.
222-
const buildDir = path.resolve('./build2');
222+
const buildDir = path.resolve('./build');
223223
if (!fs.existsSync(buildDir)) {
224224
logError(
225225
'Build directory does not exist, please run `yarn build-combined` or remove the --build option.'

scripts/release/utils.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ const getCommitFromCurrentBuild = async () => {
125125
// This is important to make the build reproducible (e.g. by Mozilla reviewers).
126126
const buildInfoJSON = join(
127127
cwd,
128-
'build2',
128+
'build',
129129
'oss-experimental',
130130
'react',
131131
'build-info.json'
@@ -136,7 +136,7 @@ const getCommitFromCurrentBuild = async () => {
136136
} else {
137137
const packageJSON = join(
138138
cwd,
139-
'build2',
139+
'build',
140140
'oss-experimental',
141141
'react',
142142
'package.json'

scripts/rollup/validate/index.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ async function lint(eslint, filepaths) {
7979
async function lintEverything() {
8080
console.log(`Linting build artifacts...`);
8181

82-
const allFilepaths = await glob('build2/**/*.js');
82+
const allFilepaths = await glob('build/**/*.js');
8383

8484
const pathsByFormat = new Map();
8585
for (const filepath of allFilepaths) {

0 commit comments

Comments
 (0)