Some space-infix-ops bug

[mightyiam]

Sorry for submitting it like this, with a screenshot from the IDE. I tried creating a PR with a failing test case. That escalated quickly.

[mightyiam]

Is this fixed by now, maybe?

[macklinu]

space-infix-ops is present in the ESLint core, not this plugin. I'm not sure what the proper next step is, but I would guess that this issue should be opened in the ESLint repo. The only slightly related issue I could find to this (type annotations + space-infix-ops) is eslint/eslint#5211.

[macklinu]

I'm thinking, based on https://github.com/nzakas/eslint-plugin-typescript/issues/101#issuecomment-372155440, that we'd need to fork the space-infix-ops rule into this plugin and handle this test case.

[zuzusik]

There is also issue in parser for this: eslint/typescript-eslint-parser#486

[madbence]

I opened eslint/eslint#10922 and eslint/eslint#10923 that can possibly resolve false positives, but it means that a new rule has to be added to eslint-plugin-typescript. as an alternative, typescript-estree could parse type aliases differently, see JamesHenry/typescript-estree#17.

[madbence]

eslint/eslint#10935 fixed this i think (eslint@5.7.0), can you confirm @mightyiam ?

[mightyiam]

Confirmed with eslint@5.8.0.

[mightyiam]

Thank you, @madbence.