Missing converter: no-import-side-effect

[JoshuaKGoldberg]

💥 Missing Converter

• tslint-to-eslint-config version: 1.2.0
• typescript-eslint version: 2.3.2

TSLint Rule Name

no-import-side-effect

[JoshuaKGoldberg]

The issue here is that this package doesn't know how to convert the TSLint rule into the equivalent ESLint rule. Accepting PRs to add in a converter! 🙌

docs/Development.md has a root overview for how to get started. Within that, docs/Architecture.md#rule-converters talks about how these converters operate. Check the typescript-eslint roadmap to see suggestions on equivalent ESLint rules to convert to.

See #189 for an example of a simple rule converter that was recently added in.

[bmeverett]

I started taking a look at this. I noticed that the TSLint rule accepts a regex for paths to ignore and the ESLint accepts a glob relative to the files in the directory. Will we have to convert between the two or should we just include a notice that the path will have to be updated somehow.

[KingDarBoja]

I started taking a look at this. I noticed that the TSLint rule accepts a regex for paths to ignore and the ESLint accepts a glob relative to the files in the directory. Will we have to convert between the two or should we just include a notice that the path will have to be updated somehow.

Err I do recommend a notice on the meantime because the pattern cases could be hard to convert.