[tslint-clean-code] Missing converters #1208
Labels
area: missing converter
A rule converter should exist but doesn't yet.
good first issue
Good for newcomers; welcome aboard!
status: accepting prs
Please, send in a PR to resolve this! ✨
💥 Missing Converter
tslint-to-eslint-config
version: 2.9.1typescript-eslint
version: 7.32.0TSLint Rule Name
map
, but any ignored result of predefined function calls that doesn’t have any side effects).no-commented-out-code
: Found https://github.com/bahmutov/eslint-rules#no-commented-out-code, however this package seems to not to be updated anymore and this specific rule has several issues described in that repo and in proposed versions: eslint/eslint#604, eslint/eslint#2735no-for-each-push
: There's no direct equivalent, we could suggesteslint-plugin-github/array-foreach
, but it forbids all usages offorEach
andeslint-plugin-fp/no-mutating-methods
to forbidpush
, however it forbids all array mutable methods.Another possibility would be suggest the use of
no-restricted-syntax
:no-map-without-usage
: Just for completeness, there are some related links: eslint/eslint#12269, eslint/eslint#12980 and typescript-eslint/typescript-eslint#2509.Package Source
TSLint package: https://github.com/Glavin001/tslint-clean-code (issue asking for a ESLint port)
ESLint package: ❌
I'd like to work on this, if accepted (just let me know how I should procceed) :)
The text was updated successfully, but these errors were encountered: