perun-network / erdstall-ts-sdk

TypeScript client SDK to interact with Erdstall.
Apache License 2.0
5 stars 2 forks source link

Remove package `tsconfig-paths` and its registrations #35

Open sebastianst opened 3 years ago

sebastianst commented 3 years ago

Since we use 1. #erdstall via an imports config in package.json and 2. a paths translation in tsconfig, we should be able to remove the package tsconfig-paths and also all registrations, e.g. in the mocha test scripts (-r tsconfig-paths/register). Note that we also translate paths in the output transpilate using ttsc.

I was able to remove it in nerd-marketplace as well.

ndzik commented 1 year ago

With our current SDK structure we cannot get rid of tsconfig-paths. This plugin is necessary for distributing our SDK with relative paths instead of the absolute ones we use referenced with a #erdstall prefix. Removing tsconfig-paths would force us to incorporate a separate bundler like webpack which in turn just becomes another tool with its own configs to maintain and use.