Seikho / dockerode-ts

Dockerode wrapped in TypeScript
1 stars 2 forks source link

publish as @types/dockerode #6

Closed oferh closed 7 years ago

oferh commented 7 years ago

Hi @Seikho,

Great work, just wondering why did you publish it as a separate package and not as @types/dockerode?

Seikho commented 7 years ago

Thank you.

I have actually been intending to publish to DefinitelyTyped for a while. I just haven't had the time or motivation to. Now that it's being requested, I'll go and do it 😛

I originally needed a quick solution and I don't really mind this pattern. It couples the package version and types together which can be an issue using @types/..., typings or tsd. Especially in this case with how quickly the Docker API moves and breaks.

RLovelett commented 7 years ago

Any news or update on this?

Seikho commented 7 years ago

Just got back from holiday. I'll submit a PR to DefinitelyTyped today or tomorrow.

Seikho commented 7 years ago

https://github.com/DefinitelyTyped/DefinitelyTyped/pull/14187

Seikho commented 7 years ago

DT has merged the PR. @types/dockerode should be available soon if it isn't already.