fix(linter): align reportUnusedDisableDirectives default value

[palexandrefernandes]

Current Behavior

The NX linter:eslint executor currently runs ESLint with reportUnusedDisableDirectives set as "off" by default when the property is not defined in executor options. This is not aligned with the default ESLint behaviour which defines this property as undefined.

This lack of alignment with the current ESLint default can be problematic for some plugins, such as eslint-plugin-eslint-comments which will try to cast this option to boolean, and can cause results from running lint with ESLint CLI and the NX ESLint executor to be different.

Expected Behavior

The reportUnusedDisableDirectives is undefined by default when this options if not defined in the executor options.

Related Issue(s)

#13241

Fixes #13241

[nx-cloud-staging]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit b33dc6a. 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 11 targets

• nx affected --target=e2e --base=3f2fa6cd38403c6ede12551f1b7efc54f21d887d --head=b33dc6ae8080d24c0ccc6800d22b59c59586d985 --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox --parallel=1
• nx affected --target=test --base=3f2fa6cd38403c6ede12551f1b7efc54f21d887d --head=b33dc6ae8080d24c0ccc6800d22b59c59586d985 --parallel=1
• nx affected --target=build --base=3f2fa6cd38403c6ede12551f1b7efc54f21d887d --head=b33dc6ae8080d24c0ccc6800d22b59c59586d985 --parallel=3
• nx depcheck --no-dte
• nx affected --target=lint --base=3f2fa6cd38403c6ede12551f1b7efc54f21d887d --head=b33dc6ae8080d24c0ccc6800d22b59c59586d985 --parallel=3
• nx lint --no-dte
• nx check-lock-files --no-dte
• nx check-commit --no-dte
• nx-cloud record -- nx format:check --base=3f2fa6cd38403c6ede12551f1b7efc54f21d887d --head=b33dc6ae8080d24c0ccc6800d22b59c59586d985
• nx check-imports --no-dte
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

[vercel]

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

Name	Status	Preview	Updated
nx-dev	✅ Ready (Inspect)	Visit Preview	Nov 21, 2022 at 11:40AM (UTC)

[meeroslav]

Thank you for the PR @palexandrefernandes

[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.