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

Call for maintainers: Flow types #1208

Closed
AugustinLF opened this issue Nov 3, 2022 · 8 comments · Fixed by #1226
Closed

Call for maintainers: Flow types #1208

AugustinLF opened this issue Nov 3, 2022 · 8 comments · Fixed by #1226

Comments

@AugustinLF
Copy link
Collaborator

Describe the Feature

Currently we're maintaing flow types in the repo. We're not testing them in anyway (they could be broken), and I do see them as something not convenient for contribution, since it's most of the time missed by contributors, and for more complex types, it could even be a problem.

Flow is not targeting open source anymore, and even RN now by default suggest TS, see facebook/react-native#35165.

We could move this maintenance to the community by moving it to https://github.com/flow-typed/flow-typed, and then, if there's still a need, people could maintain it over there.

That would be a breaking change, so we'll need to time the release.

What do you think?

@mdjastrzebski
Copy link
Member

I support the idea idea. Let's drop them in v12. None of the maintainers seems to be using Flow, so it became something we try to maintain but we don't really know if it's actually working.

I think we should actually drop Flow types without moving them to FlowTyped by ourselves because we are equality unqualified to support them either here or there. We would announce that we are looking for somebody to maintain them to gauge the interest.

@AugustinLF
Copy link
Collaborator Author

That sounds ok to me, people could still copy our definitions and add them there if needed. Might be good though to have another minor release before that.

@mdjastrzebski
Copy link
Member

We will probably have at least one minor release with #1064 and maybe #1191 if that would be ready. After that I think we should do all the planned breaking changes: #1208, #1184, #1180, #1132, maybe #1130 and release them together as v14.

@thymikee
Copy link
Member

thymikee commented Nov 3, 2022

I asked on Twitter: https://twitter.com/thymikee/status/1588145699304218624. Please share if you can and let's see what at least a part of our user base thinks about it

@mdjastrzebski mdjastrzebski changed the title Move flow types to flow typed RFC: move flow types to flow typed Nov 3, 2022
@mdjastrzebski
Copy link
Member

We've got 41 answers to @thymikee survey so far:
image

Seems like around 22% users who voted still favour supporting Flow in some form. I think this percent might be inflated due to vocal minority of Flow users voicing their concern, while many TS users might ignore it as not relevant to them, but it's still somewhat sizeable part.

I was thinking about moving to Flow Typed idea, and concluded that unless we find someone dedicated to maintain the typings it will at first become a bigger burden for us due to two repos/two releases/etc, which will lead to FT typings becoming abandonware.

I suspect that finding a dedicated FT maintainer might not be easy, as the Flow community seems to shrink from day to day.

@mdjastrzebski
Copy link
Member

@AugustinLF @thymikee wdyt? Should we do something regarding flow types?

@AugustinLF
Copy link
Collaborator Author

AugustinLF commented Nov 15, 2022

It'd be nice to do it, and shouldn't be too hard, flow-typed's tooling and contribution guidelines are good. But I don't know if anyone feels like doing so^^'

I'm short on time right now and I barely can review the PRs in the repo, so I can't do it in the upcoming weeks.

@thymikee
Copy link
Member

I believe we should pause this until we find someone from the community to maintain the types. I suggest rename this to "Call for Flow types maintainer" and ideally create a corresponding issue in flow-typed repo

@mdjastrzebski mdjastrzebski changed the title RFC: move flow types to flow typed Call for maintainers: Flow types Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants