From b816fa7be83276b7ae81f5eb406178fe52b312e3 Mon Sep 17 00:00:00 2001 From: Christopher Allford <6451942+ObliviousHarmony@users.noreply.github.com> Date: Wed, 1 Nov 2023 16:39:30 -0700 Subject: [PATCH] Added Wireit: @woocommerce/admin-e2e-tests --- .pnpmfile.cjs | 4 +- packages/js/admin-e2e-tests/package.json | 53 +++++++++++++++++------- 2 files changed, 39 insertions(+), 18 deletions(-) diff --git a/.pnpmfile.cjs b/.pnpmfile.cjs index ef63fa1fcbd..949aace3b65 100644 --- a/.pnpmfile.cjs +++ b/.pnpmfile.cjs @@ -189,7 +189,7 @@ function updateWireitDependencies( lockPackages, context ) { // In order to make maintaining the list easy we use a wireit-only script named "dependencies" to keep the list up to date. // This is an automatically generated script and that we own and so we should make sure it's always as-expected. - packageFile.wireit.dependencies = { + packageFile.wireit.dependencyOutputs = { // This is needed so we can reference files in `node_modules`. allowUsuallyExcludedPaths: true, @@ -202,7 +202,7 @@ function updateWireitDependencies( lockPackages, context ) { // wireit and it will fingerprint them for us. for ( const linkedPackage of linkedPackages ) { const packageOutputs = getPackageOutputs( linkedPackage ); - packageFile.wireit.dependencies.files.push( ...packageOutputs ); + packageFile.wireit.dependencyOutputs.files.push( ...packageOutputs ); context.log( `[wireit][${ packageFile.name }] Added '${ linkedPackage.name }' Outputs` diff --git a/packages/js/admin-e2e-tests/package.json b/packages/js/admin-e2e-tests/package.json index 49ffadfa6d8..861d451010b 100644 --- a/packages/js/admin-e2e-tests/package.json +++ b/packages/js/admin-e2e-tests/package.json @@ -25,6 +25,23 @@ "!*.tsbuildinfo" ], "sideEffects": false, + "scripts": { + "build": "pnpm --filter=\"$npm_package_name...\" run build:project", + "build:project": "pnpm run /^build:project:.*$/", + "build:project:js": "wireit", + "changelog": "composer install && composer exec -- changelogger", + "lint": "pnpm run '/^lint:lang:.*$/'", + "lint:fix": "pnpm run '/^lint:fix:lang:.*$/'", + "lint:lang:js": "eslint --output-file eslint_report.json --format json src", + "lint:fix:lang:js": "eslint src --fix", + "prepack": "pnpm run build", + "watch:build": "pnpm run build:project --watch" + }, + "lint-staged": { + "*.(t|j)s?(x)": [ + "pnpm lint:fix:lang:js" + ] + }, "dependencies": { "@jest/globals": "^27.5.1", "@types/jest": "^27.5.2", @@ -56,20 +73,24 @@ "publishConfig": { "access": "public" }, - "scripts": { - "turbo:build": "tsc --project tsconfig.json", - "prepare": "composer install", - "changelog": "composer exec -- changelogger", - "build": "pnpm -w exec turbo run turbo:build --filter=$npm_package_name", - "lint": "eslint src", - "start": "tsc --project tsconfig.json --watch", - "clean": "pnpm exec rimraf tsconfig.tsbuildinfo build build-*", - "lint:fix": "eslint src --fix", - "prepack": "pnpm run clean && pnpm run build" - }, - "lint-staged": { - "*.(t|j)s?(x)": [ - "pnpm lint:fix" - ] + "wireit": { + "build:project:js": { + "command": "tsc --project tsconfig.json", + "files": [ + "tsconfig.json", + "src/**/*.{js,jsx,ts,tsx}", + "typings/**/*.ts" + ], + "dependencies": [ + "dependencyOutputs" + ], + "output": [ + "build" + ] + }, + "dependencyOutputs": { + "allowUsuallyExcludedPaths": true, + "files": [ "required" ] + } } -} +} \ No newline at end of file