Skip to content
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

TypeScript Support #37

Open
FlorianWendelborn opened this issue Jun 2, 2019 · 8 comments
Open

TypeScript Support #37

FlorianWendelborn opened this issue Jun 2, 2019 · 8 comments
Labels
feature request New feature or request

Comments

@FlorianWendelborn
Copy link

FlorianWendelborn commented Jun 2, 2019

EDIT: Please vote for for (👍) or against (👎) TypeScript support via declaration file on this comment. (It’s a "survey" to gauge interest)


I think especially a 3D engine can benefit immensely from typescript support. It reduces the amount of documentation reading that users have to do and prevents a lot of common bugs.

I’d also advocate for converting zdog to typescript internally. I’m sure it’ll be beneficial in this kind of project.

@desandro
Copy link
Member

desandro commented Jun 3, 2019

Hello! I appreciate this suggestion. I understanding that adopting TypeScript would have several significant benefits to Zdog's source code. That said, TypeScript is not my cup of tea. My flavor is vanilla. I'll have to pass on this one.

@desandro desandro closed this as completed Jun 3, 2019
@FlorianWendelborn
Copy link
Author

@desandro is it because you’re not familiar enough with typescript or because you don’t want it to be typescript? In the First case, someone could send a PR.

@desandro
Copy link
Member

desandro commented Jun 3, 2019

I am not familiar nor do I want to use TypeScript.

@natemoo-re
Copy link

Thanks for the response! It's totally understandable that TypeScript doesn't fit into your workflow. No need to adopt a stack you're not comfortable with.

Since there are a lot of TypeScript users that could benefit from this, would you be willing to accept a PR that adds a simple declaration file to this package? I have already written one based on the public API (anything documented on zzz.dog) and would be happy to maintain it going forward.

This would be the optimal solution for Zdog/TypeScript users since npm install zdog would include all the necessary files. I would do my best to address any issues opened here with a typescript label.

If this sounds like overhead that you're not comfortable with, no worries—I can publish @types/zdog to DefinitelyTyped.

@FlorianWendelborn
Copy link
Author

To extend on @natemoo-re, even non-typescript users like yourself would benefit from it as VSCode will use the definition file for auto completion, even in JavaScript projects.

@desandro desandro reopened this Jun 5, 2019
@desandro desandro added the feature request New feature or request label Jun 5, 2019
@desandro
Copy link
Member

desandro commented Jun 5, 2019

Add a 👍 reaction to this issue if you would like to see Zdog get a declaration file. Do not add +1 comments — They will be deleted.


I've re-opened the issue to gauge general interest.

@MasterCash
Copy link

personally I created my own declaration file just for my typescript project I am working on and would love to see an official one instead of my hack-together one.

@demensky
Copy link

Added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Development

No branches or pull requests

5 participants