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

[Won't Merge] Proof Of Concept: JSON Types #13

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

Conversation

chrisgervang
Copy link
Owner

@chrisgervang chrisgervang commented Jan 30, 2019

This won't be merged. This is an experiment to see how easy it would be to add additional build targets (like Protobuf.js compatible types)

@chrisgervang chrisgervang changed the base branch from rm-flow-flag to master January 30, 2019 09:15
@goodmind
Copy link

How this knows what type to serialize and deserialize?

@chrisgervang chrisgervang changed the title Proof Of Concept: JSON Types [Won't Merge] Proof Of Concept: JSON Types Jan 30, 2019
@chrisgervang
Copy link
Owner Author

@goodmind I'm just trying something here. Updated the description.

@shanshanzhu
Copy link
Collaborator

why do we wanna duplicate the scripts here?

@@ -54,6 +55,12 @@ withAllStdIn((inputBuff: Buffer) => {
codeGenResponse.addFile(thisFileFlow);
}

// Generate Json Flowtype Files
const thisFileFlow = new CodeGeneratorResponse.File();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a flag here?

@shanshanzhu
Copy link
Collaborator

If user uses pbjs, it would be nice if 'google-protobuf' can be removed from the dependency in package.json for the pbjs use cases.

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

Successfully merging this pull request may close these issues.

3 participants