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

Rename to genType #40

Closed
cristianoc opened this issue Oct 1, 2018 · 6 comments

Comments

@cristianoc
Copy link
Owner

commented Oct 1, 2018

Gradually rename the project to genType, with small extensions followed by one breaking change.

@cristianoc

This comment has been minimized.

Copy link
Owner Author

commented Oct 1, 2018

This allows to use gentypeconfig.json:
cristianoc/genFlow@8a8250d

@nikgraf

This comment has been minimized.

Copy link
Collaborator

commented Oct 1, 2018

ohh nice, so one tool to generate multiple targets?

@tutturen

This comment has been minimized.

Copy link

commented Oct 1, 2018

I would love a tool that generates types for Flow, TypeScript and vanilla JS in one go. That would make it a joy to write js libraries in Reason.

@cristianoc

This comment has been minimized.

Copy link
Owner Author

commented Oct 1, 2018

@tutturen that’s definitely doable. The Flow and TS generated code is different, but we already have 2 extensions, so it’s easy to write both files.
For Flow and vanilla, the easiest is to just generate Flow and ignore the types for vanilla. The requirement is to have eg Babel in the pipeline. But it’s simpler than having a third kind of file generated at the same time.

@cristianoc

This comment has been minimized.

Copy link
Owner Author

commented Oct 1, 2018

Branch for the final renaming: https://github.com/cristianoc/genFlow/pull/44.

@cristianoc

This comment has been minimized.

Copy link
Owner Author

commented Oct 2, 2018

Done.

@cristianoc cristianoc closed this Oct 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.