search

node-ts / bus

A typescript based enterprise service bus framework based on enterprise integration patterns
https://bus.node-ts.com/
MIT License
271 stars 25 forks source link

Upgrade class-transformer to 0.5.1 #185

Closed nandastone closed 2 years ago

nandastone commented 2 years ago

This PR bumps the class-transformer dependency from 0.3.1 -> 0.5.1.

0.3.2, 0.4.0, 0.50 and 0.51 all had "breaking changes", although they are minimal: https://github.com/typestack/class-transformer/blob/develop/CHANGELOG.md

The objective is to suppress warnings in consuming project build output, fixed in 0.3.2:

WARNING in ../../node_modules/.pnpm/class-transformer@0.3.1/node_modules/class-transformer/storage.js
Module Warning (from ../../node_modules/.pnpm/source-map-loader@3.0.1_webpack@5.74.0/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '/x-project/node_modules/.pnpm/class-transformer@0.3.1/node_modules/src/storage.ts' file: Error: ENOENT: no such file or directory, open '/x-project/node_modules/.pnpm/class-transformer@0.3.1/node_modules/src/storage.ts'
mod35 commented 2 years ago

LGTM - all test pass, I'll just see if Andrew knows of any other issues that need looking into.

adenhertog commented 2 years ago

all looks good, though this will affect the 1.x version. 0.x which is in maintenance mode will probably need the same changes

adenhertog commented 2 years ago

published to