New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add typescript definitions #64
Comments
Not that I am aware of. If you wanted to add definitions that would be great. I personally haven't done much Typescript if you want to submit an example we'll definitely add it to the readme. |
Closing for now due to inactivity on this subject. I'm also not much of a typescript user. Will reopen if it becomes a more requested feature. |
Typescript is now a pretty solid product. Having typings for feathers would be nice. |
@talensjr Maybe interested to help out? :) |
It will definitely need to be community driven. No one on the core team currently uses Typescript. If someone wants to take on the challenge I'll re-open the issue and we can slot it into a milestone. |
Yes I would like to help, I don't know how much time can I spend on it right now, but sure I will try to help out |
I will try to give this a crack and see how it goes. |
This may be fairly quick to add to https://github.com/DefinitelyTyped/DefinitelyTyped. It can probably just extend the Express definitions from the same repository and then add the definitions for the API. It probably also makes sense to add definitions for the service interface. |
I just added this to the next release which is currently set for July 15th. @omeid please let me know if you or anybody else can take this on until then. |
Hi @daffl, Unfortunately I am pretty caught up with work and can't look into this any time soon. |
I'm going to leave this open and on the milestone until it gets closer to the release date in case anybody else is interested in picking this up. It seems to be fairly doable for anybody familiar with TypeScript interfaces. |
Of course we will do everything we can to help anybody who would like to contribute TypeScript definitions but I am going to close this issue since it does not look like anybody is going to pick it up. |
I'd really like to have this as well, because webstorm can't analyse the package looking for I'm not using Typescript itself, but it's still annoying to have everything underlined |
Bundling with Babel shouldn't really make a difference. Maybe it can't statically analyze it because of the way it is added to the app. Webstorm usually also takes JSDocs into consideration (which we don't publish either though). Either way, the open issue for TypeScript definitions is #381. Apparently making TypeScript definitions is really really hard because everybody keeps asking for them but nobody seems to be able to actually make it happen |
The issue is that maintaining them requires keeping the definitions and the code base in sync which is not always obvious. |
How I can program with typescript in featherjs? |
Here's part of a Slack discussion: alehn [11:56 AM] smixx alehn [12:28 PM] harmony alehn [12:36 PM] [12:37] eddyystop (edited) [12:40] alehn [12:40 PM] alehn [12:51 PM] eddyystop [12:54] alehn [1:00 PM] [1:01] bazaglia [1:36 PM] j2l4e [1:38 PM] [1:39] teddy-error [1:40 PM]
alehn [1:40 PM] j2l4e [1:41 PM] [1:42] [1:43] |
…#64) * added support for nested password fields option * Small code tweak
…#64) * added support for nested password fields option * Small code tweak
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue with a link to this issue for related bugs. |
Has anyone asked for Typescript definitions or tried using feathers with Typescript? I'm tempted to make them but unsure if they're actually wanted.
The text was updated successfully, but these errors were encountered: