Karlis Melderis

Results 8 comments of Karlis Melderis

I had same surprise. I have plenty of "`" left behind for strings that are not templates. So I tried to set `allowTemplateLiterals` to false and sadly code was full...

Here is my example ```ts import "reflect-metadata"; import { Expose, instanceToInstance, Type } from "class-transformer"; import { IsString } from "class-validator"; class SubDto { @Expose() @IsString() str: string; } class...

it would make sense to regenerate samples example in screenshot has `overrideResponse` twice 🤔 ``` ...overrideResponse // some faker ...overrideResponse ```

I don't think merging arrays is an obvious, easy to grasp option array should be overwritten with new array

would it be possible treat `searchElement` as `unknown`? I can imagine that before `.includes` call you don't really know what that value is or should not even care it's totally...

Does it make sense to invest time in this change? I decided to create my own wrapper decorators and wait for lib to switch away from legacy decorators first. Suspect...

I'm happy to pick this up if we can agree that it make sense and @Dassderdie proposal is valid