tj / node-migrate

Abstract migration framework for node
MIT License
1.54k stars 224 forks source link

Release with TS typings? #186

Closed qqilihq closed 2 years ago

qqilihq commented 3 years ago

There's TS types available in the main branch, but unfortunately not via NPM.

Could you push this to NPM?

carnesen commented 2 years ago

At this point a year and a half after the types were merged in this repo and a year after this issue was filed, it would appear that a new release including the types is not forthcoming. I'd say the best path forward would be to publish the type definitions as @types/migrate in DefinitelyTyped and to avoid further confusion, submit an MR to this repo to remove the types.

wesleytodd commented 2 years ago

I can publish. Sorry I must have missed that, IIRC I had ping'd a few other PRs in the hopes of getting them merged. Will do it now.

wesleytodd commented 2 years ago

ok, published and I did some of the easy dep updates while I was at it. I at least got the dep updates which removed the silly npm audit reports for old mocha (which are all not "real" in that they do not effect any test scenario).

npm i migrate@1.8.0