chore: revert changes to build process (#27584)

* Revert "chore: remove Typescript dependency from app node_modules in binary (#27577)"

This reverts commit 60d4c8356c.

* Revert "chore: simplify build script (#27547)"

This reverts commit 0a86ec686e.

* Revert "chore: upgrade lerna to 6, cache build step (#26913)"

This reverts commit 9e60aeba8f.

* build everything

* [run ci]
This commit is contained in:
Adam Stone-Lord
2023-08-18 10:05:29 -04:00
committed by GitHub
parent 60d4c8356c
commit 24f1460692
31 changed files with 1107 additions and 1316 deletions

View File

@@ -32,7 +32,7 @@ mainBuildFilters: &mainBuildFilters
- 'update-v8-snapshot-cache-on-develop'
- 'chore/update_webpack_deps_to_latest_webpack4_compat'
- 'chore/bump_loaders_and_optimize_webpack'
- 'publish-binary'
- 'astone123/revert-caching'
# usually we don't build Mac app - it takes a long time
# but sometimes we want to really confirm we are doing the right thing
@@ -44,7 +44,7 @@ macWorkflowFilters: &darwin-workflow-filters
# use the following branch as well to ensure that v8 snapshot cache updates are fully tested
- equal: [ 'update-v8-snapshot-cache-on-develop', << pipeline.git.branch >> ]
- equal: [ 'chore/update_webpack_deps_to_latest_webpack4_compat', << pipeline.git.branch >> ]
- equal: [ 'chore/bump_loaders_and_optimize_webpack', << pipeline.git.branch >> ]
- equal: [ 'astone123/revert-caching', << pipeline.git.branch >> ]
- matches:
pattern: /^release\/\d+\.\d+\.\d+$/
value: << pipeline.git.branch >>
@@ -57,7 +57,7 @@ linuxArm64WorkflowFilters: &linux-arm64-workflow-filters
- equal: [ 'update-v8-snapshot-cache-on-develop', << pipeline.git.branch >> ]
- equal: [ 'chore/update_webpack_deps_to_latest_webpack4_compat', << pipeline.git.branch >> ]
- equal: [ 'chore/bump_loaders_and_optimize_webpack', << pipeline.git.branch >> ]
- equal: [ 'publish-binary', << pipeline.git.branch >> ]
- equal: [ 'astone123/revert-caching', << pipeline.git.branch >> ]
- matches:
pattern: /^release\/\d+\.\d+\.\d+$/
value: << pipeline.git.branch >>
@@ -78,7 +78,7 @@ windowsWorkflowFilters: &windows-workflow-filters
# use the following branch as well to ensure that v8 snapshot cache updates are fully tested
- equal: [ 'update-v8-snapshot-cache-on-develop', << pipeline.git.branch >> ]
- equal: [ 'chore/update_webpack_deps_to_latest_webpack4_compat', << pipeline.git.branch >> ]
- equal: [ 'chore/bump_loaders_and_optimize_webpack', << pipeline.git.branch >> ]
- equal: [ 'astone123/revert-caching', << pipeline.git.branch >> ]
- matches:
pattern: /^release\/\d+\.\d+\.\d+$/
value: << pipeline.git.branch >>
@@ -148,7 +148,7 @@ commands:
name: Set environment variable to determine whether or not to persist artifacts
command: |
echo "Setting SHOULD_PERSIST_ARTIFACTS variable"
echo 'if ! [[ "$CIRCLE_BRANCH" != "develop" && "$CIRCLE_BRANCH" != "release/"* && "$CIRCLE_BRANCH" != "publish-binary" && "$CIRCLE_BRANCH" != "chore/bump_loaders_and_optimize_webpack" ]]; then
echo 'if ! [[ "$CIRCLE_BRANCH" != "develop" && "$CIRCLE_BRANCH" != "release/"* && "$CIRCLE_BRANCH" != "publish-binary" && "$CIRCLE_BRANCH" != "astone123/revert-caching" ]]; then
export SHOULD_PERSIST_ARTIFACTS=true
fi' >> "$BASH_ENV"
# You must run `setup_should_persist_artifacts` command and be using bash before running this command
@@ -1189,6 +1189,10 @@ commands:
command: |
source ./scripts/ensure-node.sh
yarn build --scope cypress
- run:
name: Copy Re-exported NPM Packages
command: node ./scripts/post-build.js
working_directory: cli
- run:
command: ls -la types
working_directory: cli/build
@@ -1523,7 +1527,6 @@ jobs:
- run: yarn test-scripts
# make sure packages with TypeScript can be transpiled to JS
- run: yarn lerna run build-prod --stream --concurrency 4
- run: yarn build --concurrency 4
# run unit tests from each individual package
- run: yarn test
# run type checking for each individual package
@@ -1906,7 +1909,7 @@ jobs:
- restore_cached_workspace
- run:
name: Build
command: yarn build --scope @cypress/webpack-preprocessor
command: yarn workspace @cypress/webpack-preprocessor build
- run:
name: Run tests
command: yarn workspace @cypress/webpack-preprocessor test
@@ -1951,7 +1954,7 @@ jobs:
- restore_cached_workspace
- run:
name: Build
command: yarn build --scope @cypress/vue
command: yarn lerna run build --scope @cypress/vue
- store_test_results:
path: npm/vue/test_results
- store_artifacts:
@@ -1964,7 +1967,7 @@ jobs:
- restore_cached_workspace
- run:
name: Build
command: yarn build --scope @cypress/angular
command: yarn workspace @cypress/angular build
- store-npm-logs
npm-react:
@@ -1973,7 +1976,7 @@ jobs:
- restore_cached_workspace
- run:
name: Build
command: yarn build --scope @cypress/react
command: yarn workspace @cypress/react build
- run:
name: Run tests
command: yarn test
@@ -1990,7 +1993,7 @@ jobs:
- restore_cached_workspace
- run:
name: Build
command: yarn build --scope @cypress/vite-plugin-cypress-esm
command: yarn workspace @cypress/vite-plugin-cypress-esm build
- run:
name: Run tests
command: yarn test
@@ -2007,7 +2010,7 @@ jobs:
- restore_cached_workspace
- run:
name: Build
command: yarn build --scope @cypress/mount-utils
command: yarn workspace @cypress/mount-utils build
- store-npm-logs
npm-grep:
@@ -2029,7 +2032,7 @@ jobs:
resource_class: small
steps:
- restore_cached_workspace
- run: yarn build --scope create-cypress-tests
- run: yarn workspace create-cypress-tests build
npm-eslint-plugin-dev:
<<: *defaults
@@ -2046,7 +2049,8 @@ jobs:
- run:
name: Build + Install
command: |
yarn build --scope @cypress/schematic
yarn workspace @cypress/schematic build
working_directory: npm/cypress-schematic
- run:
name: Run unit tests
command: |

View File

@@ -7,7 +7,6 @@
"clean": "node ./scripts/clean.js",
"prebuild": "yarn postinstall && node ./scripts/start-build.js",
"build": "node ./scripts/build.js",
"postbuild": "node ./scripts/post-build.js",
"dtslint": "dtslint types",
"postinstall": "patch-package && node ./scripts/post-install.js",
"size": "t=\"cypress-v0.0.0.tgz\"; yarn pack --filename \"${t}\"; wc -c \"${t}\"; tar tvf \"${t}\"; rm \"${t}\";",
@@ -177,18 +176,5 @@
"nohoist": [
"@types/*"
]
},
"nx": {
"targets": {
"build": {
"outputs": [
"{projectRoot}/types",
"{projectRoot}/build"
]
}
},
"implicitDependencies": [
"@cypress/*"
]
}
}

View File

@@ -7,6 +7,7 @@
"prebuild": "rimraf dist",
"build": "rollup -c rollup.config.mjs",
"postbuild": "node ../../scripts/sync-exported-npm-with-cli.js",
"build-prod": "yarn build",
"check-ts": "tsc --noEmit",
"lint": "eslint --ext .js,.ts,.json, ."
},
@@ -58,18 +59,6 @@
"publishConfig": {
"access": "public"
},
"nx": {
"targets": {
"build": {
"inputs": [
"{workspaceRoot}/scripts"
],
"outputs": [
"{workspaceRoot}/cli/angular"
]
}
}
},
"standard": {
"globals": [
"Cypress",

View File

@@ -5,6 +5,7 @@
"main": "dist/src/main.js",
"scripts": {
"build": "yarn prepare-example && tsc -p ./tsconfig.json && node scripts/example copy-to ./dist/initial-template && yarn prepare-copy-templates",
"build-prod": "yarn build",
"prepare-example": "node scripts/example copy-to ./initial-template",
"prepare-copy-templates": "node scripts/copy-templates copy-to ./dist/src",
"test": "cross-env TS_NODE_PROJECT=./tsconfig.test.json mocha --config .mocharc.json './src/**/*.test.ts'",
@@ -22,15 +23,14 @@
"commander": "6.2.1",
"find-up": "5.0.0",
"fs-extra": "^9.1.0",
"glob": "^7.1.6",
"inquirer": "8.2.4",
"inquirer": "7.3.3",
"ora": "^5.1.0",
"recast": "0.20.4",
"semver": "7.3.7"
},
"devDependencies": {
"@types/babel__core": "^7.1.2",
"@types/inquirer": "8.2.4",
"@types/inquirer": "7.3.1",
"@types/mock-fs": "4.10.0",
"@types/node": "16.18.39",
"@types/ora": "^3.2.0",

View File

@@ -54,16 +54,5 @@
"ng-add": {
"save": "devDependencies"
},
"nx": {
"targets": {
"build": {
"outputs": [
"{projectRoot}/src/**/*.js",
"{projectRoot}/src/**/*.d.ts",
"{projectRoot}/src/**/*.js.map"
]
}
}
},
"schematics": "./src/schematics/collection.json"
}

View File

@@ -6,6 +6,7 @@
"scripts": {
"build": "tsc || echo 'built, with type errors'",
"postbuild": "node ../../scripts/sync-exported-npm-with-cli.js",
"build-prod": "yarn build",
"check-ts": "tsc --noEmit",
"watch": "tsc -w",
"lint": "eslint --ext .js,.ts,.json, ."
@@ -32,18 +33,5 @@
"bugs": "https://github.com/cypress-io/cypress/issues/new?template=1-bug-report.md",
"publishConfig": {
"access": "public"
},
"nx": {
"targets": {
"build": {
"inputs": [
"{workspaceRoot}/scripts"
],
"outputs": [
"{workspaceRoot}/cli/mount-utils",
"{projectRoot}/dist"
]
}
}
}
}

View File

@@ -88,19 +88,6 @@
"publishConfig": {
"access": "public"
},
"nx": {
"targets": {
"build": {
"inputs": [
"{workspaceRoot}/scripts"
],
"outputs": [
"{workspaceRoot}/cli/react",
"{projectRoot}/dist"
]
}
}
},
"standard": {
"globals": [
"Cypress",

View File

@@ -57,18 +57,5 @@
},
"publishConfig": {
"access": "public"
},
"nx": {
"targets": {
"build": {
"inputs": [
"{workspaceRoot}/scripts"
],
"outputs": [
"{workspaceRoot}/cli/react18",
"{projectRoot}/dist"
]
}
}
}
}

View File

@@ -7,6 +7,7 @@
"prebuild": "rimraf dist",
"build": "rollup -c rollup.config.mjs",
"postbuild": "node ../../scripts/sync-exported-npm-with-cli.js",
"build-prod": "yarn build",
"lint": "eslint --ext .js,.jsx,.ts,.tsx,.json, .",
"check-ts": "tsc --noEmit"
},
@@ -39,18 +40,5 @@
"module": "dist/cypress-svelte.esm-bundler.js",
"publishConfig": {
"access": "public"
},
"nx": {
"targets": {
"build": {
"inputs": [
"{workspaceRoot}/scripts"
],
"outputs": [
"{workspaceRoot}/cli/svelte",
"{projectRoot}/dist"
]
}
}
}
}

View File

@@ -5,6 +5,7 @@
"main": "index.js",
"scripts": {
"build": "tsc || echo 'built, with type errors'",
"build-prod": "tsc || echo 'built, with type errors'",
"check-ts": "tsc --noEmit",
"cypress:run": "yarn cypress:run-cypress-in-cypress node ../../scripts/cypress run --project . --browser chrome",
"cypress:run-cypress-in-cypress": "cross-env CYPRESS_INTERNAL_E2E_TESTING_SELF_PARENT_PROJECT=1 HTTP_PROXY_TARGET_FOR_ORIGIN_REQUESTS=http://localhost:4455 CYPRESS_REMOTE_DEBUGGING_PORT=6666 TZ=America/New_York",
@@ -45,14 +46,5 @@
"module": "dist/index.js",
"publishConfig": {
"access": "public"
},
"nx": {
"targets": {
"build": {
"outputs": [
"{projectRoot}/dist"
]
}
}
}
}

View File

@@ -5,6 +5,7 @@
"main": "index.js",
"scripts": {
"build": "tsc || echo 'built, with type errors'",
"build-prod": "tsc || echo 'built, with type errors'",
"check-ts": "tsc --noEmit",
"cypress:open": "node ../../scripts/cypress open --project . --component",
"cypress:run": "node ../../scripts/cypress run --project . --component",
@@ -41,14 +42,5 @@
"module": "dist/index.js",
"publishConfig": {
"access": "public"
},
"nx": {
"targets": {
"build": {
"outputs": [
"{projectRoot}/dist"
]
}
}
}
}

View File

@@ -4,6 +4,7 @@
"description": "Browser-based Component Testing for Vue.js with Cypress.io ✌️🌲",
"main": "dist/cypress-vue.cjs.js",
"scripts": {
"build-prod": "yarn build",
"cy:open": "node ../../scripts/cypress.js open --component --project ${PWD}",
"cy:run": "node ../../scripts/cypress.js run --component --project ${PWD}",
"build": "rimraf dist && rollup -c rollup.config.mjs",
@@ -79,18 +80,5 @@
},
"publishConfig": {
"access": "public"
},
"nx": {
"targets": {
"build": {
"inputs": [
"{workspaceRoot}/scripts"
],
"outputs": [
"{workspaceRoot}/cli/vue",
"{projectRoot}/dist"
]
}
}
}
}

View File

@@ -7,6 +7,7 @@
"check-ts": "tsc --noEmit",
"build": "rimraf dist && yarn rollup -c rollup.config.mjs",
"postbuild": "node ../../scripts/sync-exported-npm-with-cli.js",
"build-prod": "yarn build",
"test": "echo \"Tests for @cypress/vue2 are run from system-tests\"",
"watch": "yarn build --watch --watch.exclude ./dist/**/*",
"lint": "eslint --ext .js,.jsx,.ts,.tsx,.json,.vue .",
@@ -48,18 +49,5 @@
"module": "dist/cypress-vue2.esm-bundler.js",
"publishConfig": {
"access": "public"
},
"nx": {
"targets": {
"build": {
"inputs": [
"{workspaceRoot}/scripts"
],
"outputs": [
"{workspaceRoot}/cli/vue2",
"{projectRoot}/dist"
]
}
}
}
}

View File

@@ -6,6 +6,7 @@
"scripts": {
"prebuild": "rimraf dist",
"build": "tsc || echo 'built, with type errors'",
"build-prod": "yarn build",
"check-ts": "tsc --noEmit",
"dev": "tsc --watch",
"clean": "rimraf dist",
@@ -56,14 +57,5 @@
"bugs": "https://github.com/cypress-io/cypress/issues/new?template=1-bug-report.md",
"publishConfig": {
"access": "public"
},
"nx": {
"targets": {
"build": {
"outputs": [
"{projectRoot}/dist"
]
}
}
}
}

View File

@@ -6,6 +6,7 @@
"main": "dist",
"scripts": {
"build": "rimraf dist && tsc || echo 'built, with errors'",
"build-prod": "yarn build",
"deps": "deps-ok && dependency-check --no-dev .",
"secure": "nsp check",
"semantic-release": "semantic-release",
@@ -71,14 +72,5 @@
"cypress-plugin",
"cypress-preprocessor",
"webpack"
],
"nx": {
"targets": {
"build": {
"outputs": [
"{projectRoot}/dist"
]
}
}
}
]
}

View File

@@ -5,7 +5,6 @@
"options": {
"cacheableOperations": [
"lint",
"build",
"check-ts"
],
"accessToken": "ZmNlNjA0YzAtNTM1NS00MDIwLWFlMWItNWYxYzNiMjQ4N2VkfHJlYWQtb25seQ=="
@@ -37,9 +36,7 @@
}
},
"namedInputs": {
"sharedGlobals": [
"{workspaceRoot}/scripts"
],
"sharedGlobals": [],
"default": [
"{projectRoot}/**/*",
"sharedGlobals"

View File

@@ -14,7 +14,8 @@
"binary-zip": "node ./scripts/binary.js zip",
"binary-package": "cross-env NODE_OPTIONS=--max_old_space_size=8192 node ./scripts/binary.js package",
"check-binary-on-cdn": "node ./scripts/binary.js checkIfBinaryExistsOnCdn",
"build": "lerna run build --stream",
"build": "yarn build-npm-modules && lerna run build --stream --no-bail --ignore create-cypress-tests --ignore cypress --ignore \"'@packages/{runner}'\" --ignore \"'@cypress/{angular,react,react18,vue,vue2,mount-utils,svelte}'\" && node ./cli/scripts/post-build.js && lerna run build --stream --scope create-cypress-tests",
"build-npm-modules": "lerna run build --scope cypress --scope @cypress/mount-utils --scope @cypress/react && lerna run build --scope \"'@cypress/{angular,react18,vue,vue2,svelte}'\"",
"build-prod": "lerna run build-prod-ui --stream && lerna run build-prod --stream --ignore create-cypress-tests && node ./cli/scripts/post-build.js && lerna run build-prod --stream --scope create-cypress-tests --scope",
"build-v8-snapshot-dev": "node --max-old-space-size=8192 tooling/v8-snapshot/scripts/setup-v8-snapshot-in-cypress.js --env=dev",
"build-v8-snapshot-prod": "node --max-old-space-size=8192 tooling/v8-snapshot/scripts/setup-v8-snapshot-in-cypress.js",
@@ -171,10 +172,10 @@
"http-server": "0.12.3",
"human-interval": "1.0.0",
"husky": "7.0.2",
"inquirer": "8.2.4",
"inquirer": "3.3.0",
"inquirer-confirm": "2.0.3",
"lazy-ass": "1.6.0",
"lerna": "6.6.2",
"lerna": "5.4.3",
"lint-staged": "11.1.2",
"listr2": "3.8.3",
"lodash": "^4.17.21",
@@ -206,7 +207,7 @@
"through2": "^4.0.2",
"tree-kill": "1.2.2",
"ts-node": "^10.9.1",
"typescript": "4.7.4",
"typescript": "^4.7.4",
"yarn-deduplicate": "3.1.0"
},
"engines": {
@@ -273,7 +274,6 @@
"**/ua-parser-js": "0.7.33",
"@typescript-eslint/eslint-plugin": "4.18.0",
"sharp": "0.29.3",
"typescript": "4.7.4",
"vue-template-compiler": "2.6.12"
}
}

View File

@@ -1 +0,0 @@
src/**/*.js

View File

@@ -5,6 +5,7 @@
"main": "index.js",
"scripts": {
"build": "node ./bin/cypress-electron --install",
"build-prod": "yarn build",
"clean-deps": "rimraf node_modules",
"postinstall": "echo '@packages/electron needs: yarn build'",
"start": "./bin/cypress-electron",

View File

@@ -10,6 +10,7 @@
"test-e2e": "cypress run",
"test-unit": "echo 'no unit tests'",
"build": "node ./bin/build.js && gulp build",
"build-prod": "yarn build",
"predeploy": "yarn build",
"deploy": "gh-pages -d build -b gh-pages"
},
@@ -27,16 +28,5 @@
"cypress",
"lib"
],
"types": "index.d.ts",
"nx": {
"targets": {
"build": {
"outputs": [
"{projectRoot}/cypress",
"{projectRoot}/app",
"{projectRoot}/build"
]
}
}
}
"types": "index.d.ts"
}

View File

@@ -5,6 +5,7 @@
"main": "index.js",
"scripts": {
"build": "gulp build",
"build-prod": "yarn build",
"clean": "gulp clean",
"clean-deps": "rimraf node_modules",
"postinstall": "echo '@packages/extension needs: yarn build'",

View File

@@ -6,6 +6,7 @@
"main": "dist/require.js",
"scripts": {
"build": "tsc",
"build-prod": "yarn build",
"check-ts": "tsc --noEmit && yarn -s tslint",
"clean-deps": "rimraf node_modules",
"clean": "rimraf dist",

View File

@@ -1 +0,0 @@
lib/**/*.js

View File

@@ -1 +0,0 @@
lib/**/*.js

View File

@@ -1 +0,0 @@
src/**/*.js

View File

@@ -1 +0,0 @@
lib/**/*.js

View File

@@ -7,6 +7,7 @@
"browser": "src/browser.ts",
"scripts": {
"build": "tsc",
"build-prod": "yarn build",
"check-ts": "tsc --noEmit && yarn -s tslint",
"clean": "rimraf dist",
"test": "yarn test-unit",

View File

@@ -6,6 +6,7 @@
"main": "dist/snapshot-require.js",
"scripts": {
"build": "tsc",
"build-prod": "yarn build",
"check-ts": "tsc --noEmit && yarn -s tslint",
"clean-deps": "rimraf node_modules",
"clean": "rimraf dist",

View File

@@ -242,8 +242,6 @@ const buildEntryPointAndCleanup = async (buildAppDir) => {
path.join(buildAppDir, '**', 'plist', 'dist'),
// Remove yarn locks
path.join(buildAppDir, '**', 'yarn.lock'),
// Remove Typescript dependency
path.join(buildAppDir, '**', 'node_modules', 'typescript'),
], { force: true })
// 7. Remove any empty directories as a result of the rest of the cleanup

View File

@@ -102,11 +102,6 @@ export async function buildCypressApp (options: BuildCypressAppOpts) {
if (!keepBuild) {
log('#buildPackages')
await execa('yarn', ['lerna', 'run', 'build', '--ignore', 'cli', '--concurrency', '4'], {
stdio: 'inherit',
cwd: CY_ROOT_DIR,
})
await execa('yarn', ['lerna', 'run', 'build-prod', '--ignore', 'cli', '--concurrency', '4'], {
stdio: 'inherit',
cwd: CY_ROOT_DIR,

2177
yarn.lock

File diff suppressed because it is too large Load Diff