meteor-job-collection icon indicating copy to clipboard operation
meteor-job-collection copied to clipboard

added typings file for typescript projects

Open mibto opened this issue 7 years ago • 5 comments

this file is helpful for all typescript users. I created it because i got the same error than: https://github.com/vsivsi/meteor-job-collection/issues/232

mibto avatar Jul 04 '17 12:07 mibto

Hi, thanks for this. I'm not a Typescript user but just looking at it I can tell that you put a nontrivial amount of work into those declarations!

My main concern with this is that it becomes another thing that needs to be kept up-to-date as the package evolves. But there are no tests to somehow verify or flag when that may be have become an issue. (I understand that actually doing that without replicating all of the unit tests in Typescript may be difficult).

Also, this package is highly dependent on the meteor-job npm package, and a lot of what is defined here perhaps belongs there (either in addition to, or instead of here).

I don't know enough about Typescript to judge the best way to divvy the declarations up between the two packages...

vsivsi avatar Jul 05 '17 17:07 vsivsi

Hi :) sorry i was in holidays for 2 weeks. i see your point about the maintenance. An other way would be that i publish the typings on https://github.com/DefinitelyTyped/DefinitelyTyped so that everybody can install and update this typing definition from there. If you like this idea more, then let me know and maybe you can add a word about it in your readme.

mibto avatar Jul 17 '17 09:07 mibto

At first glance, DefinitelyTyped seems like the way to go. Any drawbacks to that approach that you can think of?

vsivsi avatar Jul 17 '17 18:07 vsivsi

sounds good for me. i'll let you know when it's done.

mibto avatar Jul 20 '17 07:07 mibto

Does it have a solution at this point?

pedrofg avatar Mar 01 '18 14:03 pedrofg