fix(linter): remove leftovers after tslint support removal

[Phillip9587]

removes tslint references and generators options missed in #12279

Current Behavior

Expected Behavior

Related Issue(s)

Fixes #

[nx-cloud-staging]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 69cd720. 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 12 targets

• nx affected --target=build --base=8c8c29cdb5d172f6599b712320164a3438ae7621 --head=69cd720951ae61c580bb5341f00499254039f965 --parallel=3
• nx affected --target=e2e --base=8c8c29cdb5d172f6599b712320164a3438ae7621 --head=69cd720951ae61c580bb5341f00499254039f965 --exclude=e2e-make-angular-cli-faster,e2e-detox,e2e-js,e2e-next,e2e-workspace-create,e2e-nx-run,e2e-nx-misc,e2e-react,e2e-web,e2e-angular-extensions,e2e-angular-core,e2e-nx-plugin,e2e-cypress,e2e-node,e2e-linter,e2e-jest,e2e-add-...
• nx affected --target=e2e --base=8c8c29cdb5d172f6599b712320164a3438ae7621 --head=69cd720951ae61c580bb5341f00499254039f965 --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox,e2e-make-angular-cli-faster --parallel=1
• nx affected --target=lint --base=8c8c29cdb5d172f6599b712320164a3438ae7621 --head=69cd720951ae61c580bb5341f00499254039f965 --parallel=3
• nx affected --target=test --base=8c8c29cdb5d172f6599b712320164a3438ae7621 --head=69cd720951ae61c580bb5341f00499254039f965 --parallel=1
• nx-cloud record -- yarn check-commit
• nx-cloud record -- yarn check-imports
• nx-cloud record -- yarn check-lock-files
• nx-cloud record -- yarn depcheck
• nx-cloud record -- yarn documentation
• nx-cloud record -- yarn nx format:check --base=8c8c29cdb5d172f6599b712320164a3438ae7621 --head=69cd720951ae61c580bb5341f00499254039f965
• nx-cloud record -- yarn nx workspace-lint

---

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	Sep 30, 2022 at 9:12AM (UTC)

[Coly010]

Hey @Phillip9587! Thanks for the PR 🎉

I have some concerns with it in its current state.

The "none" option for linting has been added to a lot of generators that previously didn't have this option at all. These generators may or may not have support for handling this correctly.

It's also not entirely in line with "just" the removal of TSLint. If possible, it would be great to keep PRs to a single concern.

With that said, would it be possible for you to remove those changes and place them in a separate PR? If there's more work than you have time available to check if those generators support "none" as an option, feel free to create a Draft PR with those initial changes, and then I can takeover and finish it up.

Thanks again for this PR! It seems my IDE search didn't catch all "tslint" references 😞

I've added a "here" comment to all the generators so it's easy to find them 🙂

[Phillip9587]

The "none" option for linting has been added to a lot of generators that previously didn't have this option at all. These generators may or may not have support for handling this correctly.

I wanted to suggest adding support for the 'none' option in the @nrwl/linter generators directly. I will open a seperate PR tomorrow.

I should have time tomorrow to cleanup this PR.

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