Skip to content

Please reconsider owning your own typings. #967

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

Closed
andrew-buckley opened this issue Jun 27, 2018 · 6 comments
Closed

Please reconsider owning your own typings. #967

andrew-buckley opened this issue Jun 27, 2018 · 6 comments

Comments

@andrew-buckley
Copy link

Typescript is becoming more and more prevalent in the development community. Large library maintainers/organizations are starting to realize this and are bringing in typings into their own repositories rather than having thing hang around unmonitored on DT. By doing this, you avoid alienating TS (and JS developers who value typings) because there is much more assurance that the typings that ship with a new release are as stable as the released library. I know this has been revisited a couple times, but I'm asking again, from the bottom of the hearts of the collective React/Redux/TS community, to please reconsider owning the typings for this helpful library. This is a very popular JS library. Redux ships with typings, I think it's a fair request that this library seriously considers doing the same.

Currently there are 71 open issues related to the react-redux typings on DT.

Thanks for considering!

@markerikson
Copy link
Contributor

Neither Tim nor I are TS experts (he's used it a bit, I haven't used TS at all), and we don't have the expertise needed to maintain them. In addition, the last couple of releases we've shipped for Redux and React-Redux that have had typings changes have led to a slew of issues being filed complaining about them, particularly complaints that the changes to typings in Redux-Thunk 2.3 constitute breaking changes.

I can't do anything to maintain those typings, Tim can't do much, and frankly it's too much of a hassle. We have other things we need to be focusing on. My impression is that it makes a lot more sense to let the community handle them on DefinitelyTyped so they can be updated at their own pace.

@andrew-buckley
Copy link
Author

Thank you for the quick response, @markerikson. I can definitely understand that hesitation. I suppose all I can ask is that the maintainers at least consider some of the upsides rather than strictly the reasons for not doing it. It seems like this community has been asking for this for the past 2 years. I get it, ultimately it's your decision, but I wanted to bring this to you guys again in case you had reconsidered since the last reconsideration 6 months ago 😛 .

@andrew-buckley
Copy link
Author

And completely aside from this discussion on whether or not typings will go into this library. I encourage you to give TypeScript a shot! You might fall in love!

@markerikson
Copy link
Contributor

markerikson commented Jun 28, 2018

I actually think there's definite potential benefits in using either TS or Flow. But, neither of the apps I build at work are in a position to fully add a type layer on top atm. I briefly experimented with Flow last year, just long enough to get a feel for what adding it might be like. However, given our current toolchain and codebase, TS is not a likely option, and I don't have any side projects atm either. So, no, I probably won't be actually trying to use TS myself any time soon.

As for your previous question: can you point to some of these discussions you're referring to? This is the first I've heard any of this. (But, then again, I've deliberately avoided all of the TS-related threads for a long time now, because I don't have anything useful to add and far too much else on my plate.)

@andrew-buckley
Copy link
Author

Sure. Some are less directly related than others, but this is what I have read through.

I imagine requests for something like this won't stop coming in.

@markerikson
Copy link
Contributor

Hmm. Thanks for collecting those links.

That said, I think these prior comments from Tim and myself still apply:

#815 (comment)

Other than taking on ownership, I also doubt the benefits of keeping them here. It causes pain when TS moves forward and we don't. Releases are held to our schedule, which doesn't usually align with TS's schedule. I'd argue against adding this burden.

#815 (comment)

True. I have no connectivity to the TS world. But at the same time, that's an argument for not keeping them here. None of the maintainers here have any experience with TS. We're not able to update them here or on DT. Moving them will not help with update lag.

#799 (comment)

I get that typing is a big deal to a lot of people, but our core concern is a library that works as JS code. Redux and React-Redux have a lot of dynamic behavior, and I don't think there's any easy solutions for typing most of that.

To be frank, my personal opinion is that a lot of people spend too much time obsessing over getting typings exactly right. I understand the desire of having universal solid type coverage for an app, and it's certainly not a bad thing, but to me the time spent going for that last 10% of type coverage seems to be way in excess of the actual benefits.

Anyway, unless @timdorr has greatly changed his stance on this, I don't see us moving towards handling the typings here with the lib.

My personal priorities as a maintainer for the near future revolve around reworking our implementation to cooperate with async React behavior, not trying to spend hours learning the nuances of TS and Flow.

Thanks for the discussion, and hopefully I've clarified why this isn't something we want to do right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants