Sébastien Mischler

Results 59 comments of Sébastien Mischler

@sindresorhus If you want I can fix this before v3 (#387) because if we change the return to integrate the indexes it becomes a bracking change.

@natalya-kim @sindresorhus Seems to work with `Except` [Playground](https://www.typescriptlang.org/play?ts=4.2.3#code/JYWwDg9gTgLgBDAnmApnA3gUQB4GMVgwA0cAsilAOYo64A2ArgM7ABuKAvnAGZQQhwARElQBabiiYxBAbgBQckWgBiECHAC8ZCtVqMW7ADzoAhgC44AIzV0UJgHYcS6SxesRbDjgD44AMgxcNxs7R3kAenCMOUi4OJMAfgsGewATFG5gexRUiKi41ysQhzy4uCCij1DSrgAfaNiCpLgU9Mzs3Jj8uHNKz3tSuIr3fpqFJThVCAB5EGB4LVn5wymSAHJcNe9S9C6yxOCqhzh61oysnMGrZpHQk5a0846xxWQVNVoCBbhPwhW1EiCXCCbx7XaNHrNM7tS57AqHUZ7OoNbqWKGPGGdCG9W4lJEKXAQexSHhqJYwCxTcmaaLxCwwKAMFBEOTwhCMlByDgEokk7gfPBfSkC-CEGm7OnspkstkMplcuRAA).

@sindresorhus The PR #263 introduced in v2.3.2 fix the issue with `Expect` and `MergeExclusive` ([Playground](https://www.typescriptlang.org/play?ts=4.2.3#code/JYWwDg9gTgLgBDAnmApnA3gUQB4GMVgwA0cAsilAOYo64A2ArgM7ABuKAvnAGZQQhwARElQBabiiYxBAbgBQckWgBiECHAC8ZCtVqMW7ADzoAhgC44AIzV0UJgHYcS6SxesRbDjgD44AMgxcNxs7R3kAenCMOUi4OJMAfgsGewATFG5gexRUiKi41ysQhzy4uCCij1DSrgAfaNiCpLgU9Mzs3Jj8uHNKz3tSuIr3fpqFJThVCAB5EGB4LVn5wymSAHJcNe9S9C6yxOCqhzh61oysnMGrZpHQk5a0846xxWQVNVoCBbhPwhW1EiCXCCbZ7XaNHrNM7tS57AqHUZ7OoNbqWKGPGGdCG9W4lJEKXAQexSHhqJYwCxTcmaaLxCwwKAMFBEOTwhCMlByDjyOSE4nwbgfPBfSlC-CEGm7OnspkstkMplcnmxABqFBYRLghjgACYAHQAZj1OrgAAsYDAwEwzJFKPNTQxLHrCSBwiw0lBJNAHUxwkpxJIYOEwAw6HRwjqAGwG16oODTOipX4wQzTSwAKxQuBgABU3iQANIoRBMPNxlDYGAoNJMOAAa2LEG48YzWdzb18WgACsBcHXU63s2XmT88Ix0oYG4gmy3M0P83AiyXh95QbH3jNE8maQmk8K-qshMC13ySYLN3vxRTJmSt-vvlKevSOXKrM-FdyCUSmFU9XQIJQAAU57kiQ57JmBt6Xl8ACUMhAA)) and since `Omit` does not support it natively, there is nothing we can do, then...

@sindresorhus The only one I think now is to fix #382 i want to introduce `PickIndexSignature` the conter part of existing `RemoveIndexSignature` and for consistency I prefer to use Pick/Omit...

Sorry I spoke too fast! ~~For me it's just a tuple...:~~

I really like this idea, I see quite a few cases in internal type management, especially in those that are recursive. If you want I can try to do it!?

@valkum This is the expected behaviour, you have to use [UnwrapOpaque](https://github.com/sindresorhus/type-fest/blob/main/source/opaque.d.ts#L104) to get the base type before passing it to another type.

I am closing this issue, it seems impossible to reproduce the error with the old or new version. Feel free to reopen this issue if you have a reproducible example.

> I would propose reverting #65 and moving back to: > > ```ts > export type JsonObject = {[key: string]: JsonValue}; > ``` FI: A version that supports both cases...

@xenoterracide Please provide a minimal [working exemple](https://www.typescriptlang.org/play?#code/) with actual and expected results.