Skip to content

Revert "Build: Migrate repo to Nx 20"#29820

Merged
yannbf merged 1 commit into
nextfrom
revert-29807-nx-20
Dec 5, 2024
Merged

Revert "Build: Migrate repo to Nx 20"#29820
yannbf merged 1 commit into
nextfrom
revert-29807-nx-20

Conversation

@yannbf

@yannbf yannbf commented Dec 5, 2024

Copy link
Copy Markdown
Member

Reverting the PR because CI started to fail.

Reverts #29807

Greptile Summary

This PR reverts the migration to Nx 20 due to CI failures, restoring configurations and dependencies back to Nx 18.0.6 across the codebase.

  • Restored code/migrations.json with Nx 17.3.0 and 18.0.0 migration configurations
  • Downgraded Nx packages from 20.1.4 to 18.0.6 in code/package.json and scripts/package.json
  • Reverted nx.json to include previous cacheDirectory and cli settings
  • Removed .nx/workspace-data ignore patterns from .gitignore and .prettierignore
  • Restored previous project configurations in framework projects like angular and vue3

@yannbf yannbf merged commit d908610 into next Dec 5, 2024
@yannbf yannbf deleted the revert-29807-nx-20 branch December 5, 2024 10:04
@yannbf yannbf added the build Internal-facing build tooling & test updates label Dec 5, 2024

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

12 file(s) reviewed, 5 comment(s)
Edit PR Review Bot Settings | Greptile

Comment thread code/.gitignore

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style: Removing this file will cause vite.config timestamp files to be tracked in git. Consider keeping the ignore rule by moving it to the root .gitignore instead.

Comment thread .gitignore
Comment on lines 57 to 59
code/.nx/cache
code/.nx/workspace-data
code/.vite-inspect
.nx/cache

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style: consider keeping .nx/workspace-data ignored to prevent committing temporary Nx workspace data

Comment thread code/nx.json
{
"$schema": "./node_modules/nx/schemas/nx-schema.json",
"extends": "nx/presets/npm.json",
"cacheDirectory": "../.nx/cache",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style: cacheDirectory is deprecated in newer Nx versions and should be removed. It defaults to .nx/cache in the workspace root.

Comment thread code/nx.json
Comment on lines +8 to +10
"affected": {
"defaultBase": "next"
},

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style: affected.defaultBase is redundant with the top-level defaultBase setting on line 6

Comment thread code/nx.json
"production": [
"default",
"!{projectRoot}/src/**/**/*.{test,spec,stories}.?(c|m)[jt]s?(x)?(.snap)",
"!{projectRoot}/src/**/**/*.{test,spec,stories}.?(c|m)[jt]s?(x)?(.snap),",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

logic: invalid glob pattern - trailing comma inside the pattern will cause file matching issues

@github-actions

github-actions Bot commented Dec 5, 2024

Copy link
Copy Markdown
Contributor
Fails
🚫

PR is not labeled with one of: ["ci:normal","ci:merged","ci:daily","ci:docs"]

Generated by 🚫 dangerJS against b713e8e

@github-actions github-actions Bot mentioned this pull request Dec 5, 2024
11 tasks
@nx-cloud

nx-cloud Bot commented Dec 5, 2024

Copy link
Copy Markdown

☁️ Nx Cloud Report

CI is running/has finished running commands for commit b713e8e. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Internal-facing build tooling & test updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant