Daniel Nixon
Daniel Nixon
I may have caused this in https://github.com/agiledigital/typed-redux-saga/issues/13
Could you formulate this as a dtslist `$ExpectType` failure in index.test.ts?
PRs welcome
Thanks @ivan-aksamentov that's a good insight. I wonder if there's something we can do with eslint to catch that 🤔
That might also explain #27
See https://github.com/danielnixon/eslint-config-typed-fp/blob/master/src/index.ts#L46-L65
https://github.com/anyhowstep/ts-eslint-invariant-mutable-properties looks related
This rule should recommend https://immutable-js.github.io/immutable-js/docs/#/Map or TypeScript's `ReadonlyMap` type if the builtin Map _really_ must be used.
Thanks for the kind words @szagi3891 . What you describe is a good catch (thanks!) but it's a separate issue to the one in this ticket. I've created https://github.com/danielnixon/eslint-plugin-total-functions/issues/68 for...
This just landed a few hours ago: https://github.com/typescript-eslint/typescript-eslint/commit/d7dc108580cdcb9890ac0539e7223aedbff4a0ed Maybe the best approach here is to teach that rule a new `never` option (akin to https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/consistent-type-assertions.md#assertionstyle)