Skip to content
This repository has been archived by the owner on Nov 2, 2018. It is now read-only.

Add 'npm run update-schema' shortcut #4

Merged
merged 1 commit into from
Aug 13, 2015

Conversation

cletusw
Copy link
Contributor

@cletusw cletusw commented Aug 12, 2015

Running './scripts/updateSchema.js' only works if you have babel-node installed globally.
'npm run', on the other hand, automatically includes installed node modules in the PATH.

Running './scripts/updateSchema.js' only works if you have babel-node installed globally.
'npm run', on the other hand, automatically includes installed node modules in the PATH.
@zpao
Copy link
Contributor

zpao commented Aug 12, 2015

Makes sense to me, will let @steveluscher make the final call.

@steveluscher
Copy link
Contributor

One day, we'll automate this so you don't even have to think about regenerating the schema. For now, this is great!

Would you like to apply this treatment to the examples in the main Relay repository too? https://github.com/facebook/relay/tree/master/examples

steveluscher added a commit that referenced this pull request Aug 13, 2015
Add 'npm run update-schema' shortcut
@steveluscher steveluscher merged commit 2ea1a48 into facebookarchive:master Aug 13, 2015
@cletusw
Copy link
Contributor Author

cletusw commented Aug 13, 2015

Sure thing. I'll get right on it.

Em qua, 12 de ago de 2015 às 18:34, Steven Luscher notifications@github.com
escreveu:

One day, we'll automate this so you don't even have to think about
regenerating the schema. For now, this is great!

Would you like to apply this treatment to the examples in the main Relay
repository too? https://github.com/facebook/relay/tree/master/examples


Reply to this email directly or view it on GitHub
#4 (comment)
.

@cletusw
Copy link
Contributor Author

cletusw commented Aug 13, 2015

@steveluscher Done. facebook/relay#47

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

Successfully merging this pull request may close these issues.

4 participants