Adornis/typescript

get rid of barbatus:typescript-runtime

Closed this issue · 3 comments

Package.js references barbatus:typescript-runtime

api.imply('barbatus:typescript-runtime@1.1.0');

This module purpose seem to only wrap around TypeScript tslib package. Likely this npm package should be included in the typescript-compiler module (unless I am missing something here.)

yorrd commented

@PopGoesTheWza so what you're proposing is including tslib in the dependencies directly and removing the dependency to barbatus? I'm not exactly sure what the package is doing and how it works atm. Do you feel up to writing a pull request with those changes?

@yorrd the barbatus:typescript-runtime module only add the tslib package.
This tslib package provides transpiled typescript helpers to improve performance.

This should be better used if the tslib package is directly added in the adornis:typescript-compiler module.

This can be closed now that we instead rely on
"importHelpers": true,
in tsconfig