Skip to content

Make sure writtenFiles is propagates#682

Merged
jbaxleyiii merged 1 commit intoapollographql:masterfrom
bobvanderlinden:patch-1
Nov 12, 2018
Merged

Make sure writtenFiles is propagates#682
jbaxleyiii merged 1 commit intoapollographql:masterfrom
bobvanderlinden:patch-1

Conversation

@bobvanderlinden
Copy link
Copy Markdown
Contributor

Currently whenever client:codegen is used the command outputs the following:

 ✔ Generating query files with 'typescript' target - wrote undefined files

undefined is because writtenFiles is undefined. writtenFiles is undefined because write doesn't return any value. generate returns the number of written files, so that should be returned in write as well.

Currently whenever `client:codegen` is used the command outputs the following:

```
 ✔ Generating query files with 'typescript' target - wrote undefined files
```

`undefined` is because `writtenFiles` is `undefined`. `writtenFiles` is `undefined` because `write` doesn't return any value. `generate` returns the number of written files, so that should be returned in `write` as well.
@apollo-cla
Copy link
Copy Markdown

@bobvanderlinden: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Meteor Contributor Agreement here: https://contribute.meteor.com/

@bobvanderlinden
Copy link
Copy Markdown
Contributor Author

Signed the agreement.

@jbaxleyiii
Copy link
Copy Markdown
Contributor

@bobvanderlinden Thanks for this! 💯

@jbaxleyiii jbaxleyiii merged commit e7283c9 into apollographql:master Nov 12, 2018
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