-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
chore(ci): Move monorepo to nx #19325
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Changes from all commits
100b47d
b2a6c5f
abb61f3
0ceab17
ab8f7cb
c646f25
9415afb
e8a9f34
6fdc170
31d3561
1923f8e
c5a2b89
44a9023
c83d7df
8ef6313
e3256fc
42703c2
7cb8127
3517d7f
45c34ab
29b9612
00e42fb
55f3f7d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -24,7 +24,6 @@ packages/*/sentry-*.tgz | |
| # logs | ||
| yarn-error.log | ||
| npm-debug.log | ||
| lerna-debug.log | ||
| local.log | ||
|
|
||
| # ide | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| { | ||
| "_comment": "Auto-generated by scripts/bump-version.js. Used by the gitflow sync workflow to detect version bumps. Do not edit manually.", | ||
| "version": "10.39.0" | ||
| } |
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,6 +10,9 @@ | |
| "production": ["default", "!{projectRoot}/test/**/*", "!{projectRoot}/**/*.md", "!{projectRoot}/*.tgz"] | ||
| }, | ||
| "targetDefaults": { | ||
| "build:dev": { | ||
| "dependsOn": ["^build:transpile", "^build:types"] | ||
| }, | ||
| "build:bundle": { | ||
| "inputs": ["production", "^production"], | ||
| "dependsOn": ["build:transpile"], | ||
|
|
@@ -58,5 +61,8 @@ | |
| } | ||
| }, | ||
| "$schema": "./node_modules/nx/schemas/nx-schema.json", | ||
| "cacheDirectory": ".nxcache" | ||
| "cacheDirectory": ".nxcache", | ||
| "tui": { | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This works reasonably well for me I have to say, which shell are you on? |
||
| "autoExit": true | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,49 +1,49 @@ | ||
| { | ||
| "private": true, | ||
| "scripts": { | ||
| "build": "node ./scripts/verify-packages-versions.js && lerna run build:transpile,build:types,build:bundle", | ||
| "build:bundle": "lerna run build:bundle", | ||
| "build:dev": "lerna run build:types,build:transpile", | ||
| "build:dev:filter": "lerna run build:dev --include-filtered-dependencies --include-filtered-dependents --scope", | ||
| "build:transpile": "lerna run build:transpile", | ||
| "build:types": "lerna run build:types", | ||
| "build:watch": "lerna run build:watch", | ||
| "build:dev:watch": "lerna run build:dev:watch", | ||
| "build:types:watch": "ts-node scripts/build-types-watch.ts", | ||
| "build": "node ./scripts/verify-packages-versions.js && nx run-many -t build:transpile build:types build:bundle", | ||
| "build:bundle": "nx run-many -t build:bundle", | ||
| "build:dev": "nx run-many -t build:types build:transpile", | ||
| "build:dev:filter": "nx run-many -t build:dev -p", | ||
chargome marked this conversation as resolved.
Show resolved
Hide resolved
sentry[bot] marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| "build:transpile": "nx run-many -t build:transpile", | ||
| "build:types": "nx run-many -t build:types", | ||
| "build:watch": "nx run-many -t build:watch", | ||
| "build:dev:watch": "nx run-many -t build:dev:watch", | ||
| "build:tarball": "run-s clean:tarballs build:tarballs", | ||
| "build:tarballs": "lerna run build:tarball", | ||
| "build:tarballs": "nx run-many -t build:tarball", | ||
| "changelog": "ts-node ./scripts/get-commit-list.ts", | ||
| "generate-changelog": "ts-node ./scripts/generate-changelog.ts", | ||
| "circularDepCheck": "lerna run circularDepCheck", | ||
| "circularDepCheck": "nx run-many -t circularDepCheck", | ||
| "clean": "run-s clean:build clean:caches", | ||
| "clean:build": "lerna run clean", | ||
| "clean:build": "nx run-many -t clean", | ||
| "clean:caches": "yarn rimraf eslintcache .nxcache .nx", | ||
| "clean:deps": "lerna clean --yes && rm -rf node_modules && yarn", | ||
| "clean:deps": "rimraf packages/*/node_modules dev-packages/*/node_modules && rm -rf node_modules && yarn", | ||
| "clean:tarballs": "rimraf {packages,dev-packages}/*/*.tgz", | ||
| "clean:watchman": "watchman watch-del \".\"", | ||
| "clean:all": "run-s clean:build clean:tarballs clean:caches clean:deps clean:watchman", | ||
| "fix": "run-s fix:oxfmt fix:lerna", | ||
| "fix:lerna": "lerna run fix", | ||
| "fix": "run-s fix:oxfmt fix:eslint", | ||
| "fix:eslint": "nx run-many -t fix", | ||
| "fix:oxfmt": "oxfmt . --write", | ||
| "format:check": "oxfmt . --check", | ||
| "format": "oxfmt . --write", | ||
| "lint": "run-s lint:oxfmt lint:lerna", | ||
| "lint:lerna": "lerna run lint", | ||
| "lint": "run-s lint:oxfmt lint:eslint", | ||
| "lint:eslint": "nx run-many -t lint", | ||
| "lint:oxfmt": "oxfmt . --check", | ||
| "lint:es-compatibility": "lerna run lint:es-compatibility", | ||
| "lint:es-compatibility": "nx run-many -t lint:es-compatibility", | ||
| "dedupe-deps:check": "yarn-deduplicate yarn.lock --list --fail", | ||
| "dedupe-deps:fix": "yarn-deduplicate yarn.lock", | ||
| "postpublish": "lerna run --stream --concurrency 1 postpublish", | ||
| "test": "lerna run --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,node-core-integration-tests,cloudflare-integration-tests}\" test", | ||
| "test:unit": "lerna run --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,node-core-integration-tests,cloudflare-integration-tests}\" test:unit", | ||
| "test:update-snapshots": "lerna run test:update-snapshots", | ||
| "postpublish": "nx run-many -t postpublish --parallel=1", | ||
| "test": "nx run-many -t test --exclude \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,node-core-integration-tests,cloudflare-integration-tests}\"", | ||
| "test:scripts": "vitest run scripts/bump-version.test.ts", | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. q: Do the tests actually run in the CI?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nope they don't, I just left them for local development as they're not properly integrated in our vite setup |
||
| "test:unit": "nx run-many -t test:unit --exclude \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,node-core-integration-tests,cloudflare-integration-tests}\"", | ||
| "test:update-snapshots": "nx run-many -t test:update-snapshots", | ||
| "test:pr": "nx affected -t test --exclude \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,node-core-integration-tests,cloudflare-integration-tests}\"", | ||
| "test:pr:browser": "UNIT_TEST_ENV=browser ts-node ./scripts/ci-unit-tests.ts --affected", | ||
| "test:pr:node": "UNIT_TEST_ENV=node ts-node ./scripts/ci-unit-tests.ts --affected", | ||
| "test:ci:browser": "UNIT_TEST_ENV=browser ts-node ./scripts/ci-unit-tests.ts", | ||
| "test:ci:node": "UNIT_TEST_ENV=node ts-node ./scripts/ci-unit-tests.ts", | ||
| "test:ci:bun": "lerna run test --scope @sentry/bun", | ||
| "yalc:publish": "lerna run yalc:publish" | ||
| "test:ci:bun": "nx run-many -t test -p @sentry/bun", | ||
| "yalc:publish": "nx run-many -t yalc:publish" | ||
| }, | ||
| "volta": { | ||
| "node": "20.19.2", | ||
|
|
@@ -127,7 +127,7 @@ | |
| "es-check": "^7.2.1", | ||
| "eslint": "8.57.0", | ||
| "jsdom": "^21.1.2", | ||
| "lerna": "8.2.4", | ||
| "nx": "22.5.0", | ||
| "madge": "8.0.0", | ||
| "nodemon": "^3.1.10", | ||
| "npm-run-all2": "^6.2.0", | ||
|
|
@@ -146,8 +146,7 @@ | |
| }, | ||
| "//_resolutions_comment": [ | ||
| "Because new versions of strip-ansi, string-width, and wrap-ansi are ESM only packages,", | ||
| "we need to resolve them to the CommonJS versions.", | ||
| "This is a temporary solution until we can upgrade to a version of lerna that supports ESM packages" | ||
| "we need to resolve them to the CommonJS versions." | ||
| ], | ||
| "resolutions": { | ||
| "gauge/strip-ansi": "6.0.1", | ||
|
|
||


Uh oh!
There was an error while loading. Please reload this page.