Migration update-16-8-0-add-ignored-files is failing

[C0ZEN]

Current Behavior

Migrating from Nx 16.7 to 16.8, and running the migration update-16-8-0-add-ignored-files is not working.

The log error is:

 >  NX   Failed to run update-16-8-0-add-ignored-files from @nx/linter. This workspace is NOT up to date!
 >  NX   Cannot find apps/{project-name}/.eslintrc.json

Note that I don't have an .eslintrc.json, but instead a .eslintrc.js.
Normally, it should be covered inside the eslint-file.ts, but there is only coverage for the json file for this migration 🤔 .

Expected Behavior

There is no error and the migration run successfully.

GitHub Repo

No response

Steps to Reproduce

1. Create a new project with version 16.7.
2. Change the .eslintrc.json to a .eslintrc.js
3. npx nx migrate nx
4. npm i
5. npx nx migrate --run-migrations

Nx Report

Node   : 18.17.1
   OS     : darwin-arm64
   npm    : 9.6.7
   
   nx                 : 16.8.0
   @nx/js             : 16.8.0
   @nx/linter         : 16.8.0
   @nx/workspace      : 16.8.0
   @nx/cypress        : 16.8.0
   @nx/devkit         : 16.8.0
   @nx/eslint-plugin  : 16.8.0
   @nx/react          : 16.8.0
   @nx/storybook      : 16.8.0
   @nrwl/tao          : 16.8.0
   @nx/vite           : 16.8.0
   @nx/web            : 16.8.0
   nx-cloud           : 16.3.0
   typescript         : 5.1.3
   ---------------------------------------
   Community plugins:
   @c0zen/nx-sonarqube : 2.6.0
   nx-stylelint        : 15.0.0

Failure Logs

>  NX   Failed to run update-16-8-0-add-ignored-files from @nx/linter. This workspace is NOT up to date!
>  NX   Cannot find apps/{project-name}/.eslintrc.json

Package Manager Version

No response

Operating System

• macOS
• Linux
• Windows
• Other (Please specify)

Additional Information

No response

[C0ZEN]

@jaysoo FYI, since you created this migration 🙏🏻

[jaysoo]

We added some utilities to update overrides in both .eslintrc.json and (new) eslint.config.js formats, however it is causing issues with other formats. I added a guard to skip other formats like .eslintrc.js so the update won't fail.

#19026

We'll cut a patch for this.

[jaysoo]

For now, you can remove the entry from migrations.json and run the migrations.

{
  "version": "16.8.0",
  "description": "update-16-8-0-add-ignored-files",
  "implementation": "./src/migrations/update-16-8-0-add-ignored-files/update-16-8-0-add-ignored-files",
  "package": "@nx/linter",
  "name": "update-16-8-0-add-ignored-files"
}

[github-actions]

This issue has been closed for more than 30 days. If this issue is still occuring, please open a new issue with more recent context.