React-vis: Typescript Definition File for the library.

Created on 11 Oct 2017  路  15Comments  路  Source: uber/react-vis

I have searched for and not found a typescript definition file (d.ts) for this library.

Is there one in the pipeline? I've created a horrible stub for now, but if I get the chance I will tidy it up and produce a PR.

enhancement help wanted

Most helpful comment

Well, it seems others have lost interest (?). I have a project where I plan to use react-vis with TypeScript. My current plan is to use @vdelacou's typings and then refine them as I need to. If I think the result is reasonable, I will submit a PR back here to fold in whatever I have back into this project. This assumes (perhaps incorrectly) that I will have the proper time to do this. I'll try, but not promises.

All 15 comments

I'd love to contribute to this if someone's working on this. Would love a Typescript def file for this library!

Any word on this? I'd be super stoked!!

still nothing?

I write a first version, but still need a lot of work (eg: delete all any and write the correct type)

https://gist.github.com/vdelacou/c7025196695148da33b3ee1a0efacdf1

@btford or @mcnuttandrew Would you accept a pull request for this? (i.e., keeping a .d.ts file in the react-vis repo)?

I would welcome a PR to this effect. I am not familiar with typescript, so it would be good if you could include some resources on how to review the PR along the PR 馃槵馃槵馃槵馃槵.

I would propose we take @vdelacou's version as a starting point and iterate on it. We could add some tests as we go based on examples, etc. to ensure the typings matched up.

@Fles, @codepressd , @zfranklyn @awjreynolds are you interested in participating?

Seems like the simplest thing would be to fork the repo and work on the typings in the fork and then submit a PR to merge it all when we are reasonably happy with it. Comments?

Well, it seems others have lost interest (?). I have a project where I plan to use react-vis with TypeScript. My current plan is to use @vdelacou's typings and then refine them as I need to. If I think the result is reasonable, I will submit a PR back here to fold in whatever I have back into this project. This assumes (perhaps incorrectly) that I will have the proper time to do this. I'll try, but not promises.

I use the typescript definition mentioned by @vdelacou and improved it a but t it still needs all lot of anys replaced. I create a merge request to add the first version to the repository #850 so that hopefully it will be improved.

Add pr to @types?

heads up @evgsil made a repo under the MIT license with the typings from that gist.
https://github.com/evgsil/react-vis-typings

I've submitted pull request https://github.com/uber/react-vis/pull/1003 that should add generate-types.js to this package. This script will generate actual typings with each react-vis build

It's probably worth mentioning here that #1003 was closed in favor of submitting some type definitions to Definitely-Typed: https://github.com/uber/react-vis/pull/1003#issuecomment-500028248

As mentioned on that thread, @fstylermiller has started a repo to work on a Definitely Typed pull request: https://github.com/fstylermiller/DefinitelyTyped

there is now a typescript definition file on definitely typed! install it now via @types/react-vis big credit to @evgsil for the types

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jonnydodad picture jonnydodad  路  4Comments

cang-afero picture cang-afero  路  4Comments

cvalmonte picture cvalmonte  路  4Comments

jckr picture jckr  路  5Comments

ZKruse picture ZKruse  路  4Comments