Reduce unnecessary depdendency burden of eslint-plugin-nx

[luxaritas]

• I'd be willing to implement this feature (contributing guide)

Description

Currently, eslint-plugin-nx relies on @nrwl/workspace, which itself relies on a number of packages including @nrwl/jest. It would be great to remove this dependency so that the transitive dependencies are no longer included

Motivation

In my specific case, I'm using "Nx Core" (not using @nrwl/workspace or any other nrwl plugins), and my environment uses Vitest rather than Jest, so I was surprised to see jest-related dependencies being installed. While this isn't a huge issue, it would be great to cut down on unneeded dependencies.

Suggested Implementation

It appears that @nrwl/workspace is being used for AST, file, and runtime lint utils. It seems reasonably to move these utils into nx and remove the @nrwl/workspace dependency.

[meeroslav]

Hey @luxaritas, I totally agree. Some of these should end up in devkit and some should go to linter.
These parts were there historically because TSLint rule is part of the workspace package.

Detaching this would require getting rid of TSLint which is planned for one of the future versions. Unfortunately, it doesn't have a high priority right now.

[luxaritas]

Out of curiosity, is there a reason why moving the implementation and then importing in (or even re-exporting from) workspace wouldn't work? I suppose that could create some internal complexity/"noise" (particularly re-exports).

[luxaritas]

Looks like tslint is now getting stripped out: #12279

[meeroslav]

Yes, we will move the above-mentioned logic from @nrwl/workspace soon!

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