fix(linter): handle non-JSON eslintrc files when updating overrides

[jaysoo]

The new ESLint overrides update logic only works if the config is:

1. .esilntrc.json -- we assume config is JSON when updating old config format
2. eslint.config.js -- we support new flat config format, but it is different from existing .eslintrc.js format

For other formats like .eslintrc.js, .eslintrc.yaml, etc., we should ignore updating the overrides entry since they are not in the expected format .

Current Behavior

Expected Behavior

Related Issue(s)

Fixes #19020

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
nx-dev	⬜️ Ignored (Inspect)	Visit Preview		Sep 6, 2023 3:37pm

[nx-cloud]

☁️ Nx Cloud Report

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

📂 See all runs for this branch

---

✅ Successfully ran 8 targets

• nx affected --target=e2e --base=2bc70310170c59b039c3b506189e17f1a9a6ce0c --head=6cbd1fc495bd9399f4ba291296181a61821b564b --parallel=1
• nx affected --target=test --base=2bc70310170c59b039c3b506189e17f1a9a6ce0c --head=6cbd1fc495bd9399f4ba291296181a61821b564b --parallel=1
• nx affected --target=build --base=2bc70310170c59b039c3b506189e17f1a9a6ce0c --head=6cbd1fc495bd9399f4ba291296181a61821b564b --parallel=3
• nx affected --target=lint --base=2bc70310170c59b039c3b506189e17f1a9a6ce0c --head=6cbd1fc495bd9399f4ba291296181a61821b564b --parallel=3
• nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• nx-cloud record -- nx format:check --base=2bc70310170c59b039c3b506189e17f1a9a6ce0c --head=6cbd1fc495bd9399f4ba291296181a61821b564b
• nx documentation --no-dte
• nx affected -t e2e-macos --parallel=1 --base=2bc70310170c59b039c3b506189e17f1a9a6ce0c --head=6cbd1fc495bd9399f4ba291296181a61821b564b

---

Sent with 💌 from NxCloud.

[jaysoo]

One of the Node e2e tested timed out in CI, bumped it from 1 minute to 5 minutes thresshold.

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.