forked from typestack/class-transformer
-
Notifications
You must be signed in to change notification settings - Fork 5
Open
Description
Is there an existing issue that is already proposing this?
- I have searched the existing issues
Is your feature request related to a problem? Please describe it
plainToClass is incorrectly named. class-transformer has since provided a properly named plainToInstance function with seemingly identical interface. Please pull in this change from upstream.
Describe the solution you'd like
Pull in plainToInstance from upstream class-transformer repo
Teachability, documentation, adoption, migration strategy
Identically to plainToClass but plainToInstance.
What is the motivation / use case for changing the behavior?
- Accuracy of function names
- Minimize difference between
class-transformerand@nestjs/class-transformer - This bug means we need to blacklist
class-transformerfrom our imports, which means either@nestjs/class-transformerupdates to include the correct interface or we revert to the inaccurate interface
Metadata
Metadata
Assignees
Labels
No labels