Explain why the typescript dependency?

[zachlysobey]

This is a nitpick for sure, but I think there is a superfluous typescript dependency?

There doesn't seem to be any .ts files in the project, no other reference to typescript.

https://github.com/blockstack/animal-kingdom/blob/master/package.json#L12

If this was added in error, probably makes sense to do:

npm uninstall --save typescript

That should get rid of package.json AND package-lock.json references.

[moxiegirl]

Hi @zachlysobey The blockstack.js lib is written in typescript. We recently did an overhaul of that package to determine the minimum required set of packages.

[zachlysobey]

@moxiegirl sounds like it is there for a reason then? But I'm still confused.

I see that typescript is a (build-time) dev-dependency of the blockstack.js project. But that shouldn't mean that you would need to include that package in consumers of blockstack.js. And certainly not as a regular run-time dependency. If typescript really is required, than that would indicate to me that something really really weird is going on :)

[Sahana-Srinivasan]

Hi @zachlysobey You're right; nothing's written in typescript in Animal Kingdom and it's not needed. I just put it in there because a lot of folks in Animal Kingdom are new users and we don't want to confuse them with a lot of peer dependency warnings, in case someone doesn't know they can be ignored (having typescript in there clears about a half-dozen).

[moxiegirl]

@zachlysobey Thank you for this issue. I'm going to go ahead and change the title slightly. I'll add a small explanation to the docs about why we do this.