Core ESLint rules not disabled when their @typescript-eslint equivalent is enabled

[JamesHenry]

🐛 Bug Report

• tslint-to-eslint-config version: 2.13.0
• ESLint version: N/A
• Node version: 16.15.0

Actual Behavior

Found when updating angular-eslint's usage of tslint-to-eslint-config

See my minimal repro, e.g. quotes and semi are both enabled despite their typescript-eslint counterparts also being present and enabled.

This is a regression because this didn't use to happen in 2.4.0 which is where I am migrating from (yes I know quite a way behind!)

Expected Behavior

quotes and semi should be explicitly be set to off

Reproduction

Minimal repro here: https://github.com/JamesHenry/tslint-to-eslint-config-repro

• First commit adds tslint and codelyzer (Angular specific TSLint rules)
• Second commit is simply the result of running npx tslint-to-eslint-config, see the generated .eslintrc.js

[JamesHenry]

This is a continuation of #1207

[neel-desh]

Is this open for contribution? if yes, can I pick it up?

[JoshuaKGoldberg]

Absolutely, yes please @neel-desh! 🚀

[JamesHenry]

How did you get on @neel-desh?

[JoshuaKGoldberg]

Sorry for taking so long on this @JamesHenry, it fell off my radar! I just tried locally with the latest, [email protected], and get an .eslintrc.js with "quotes": "off" and "semi": "off". Does this issue still repro for you?

[hyperupcall]

Since I made the PR that closed #1207, I checked this out and for me I get the expected behavior in both cases: "quotes: "off", "semi": "off", when running locally and through npx.

[JoshuaKGoldberg]

Great - in that case I'll close this out. Thanks for investigating @hyperupcall!

If anybody is experiencing this behavior as broken, please do post back here or in a new issue with full reproduction steps we can take.