-
Notifications
You must be signed in to change notification settings - Fork 103
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
Have a better way of getting the schema, for example a URL #4
Comments
First of all: Thanks for this great plugin! This is a great idea and exactly what we tried to facilitate with babel-plugin-react-relay when developing apps with Relay. We've implemented this feature using sync-request due to the limitation of Babel that you just can run synchronous plugins. I'd be very interested to learn more about the implementation details of the upcoming feature. Maybe we can improve the babel-plugin as well. |
One thing I'm really interested in is standardizing some config about how to reach your server. Maybe something like this: ardatan/graphql-tools#40 (comment) That way, our different plugins might be able to share information about the server, instead of all being configured separately. What do you think? What info would you need in there for your plugin? |
That sounds like a very good idea. The problem with Babel is that it's very limited concerning the way how you pass plugin configuration, therefore we had to provide the config in the This is also something worth considering: facebook/relay#1080 |
Yeah having support for multiple schemas is something I thought about as well. I wonder how you would specify which queries are against which schema for the linter...
That's kind of what I meant, yeah - something that sits at the root of your project and all build tools can draw from. Maybe we can start listing requirements here? https://github.com/apollostack/graphqlrc |
@schickling hey, I added you to https://github.com/apollostack/graphqlrc, so if you have time to lay down some ideas feel free to just commit to the README! |
Any progress on this? |
Not yet. @stubailo and I have to find a time to finish this once and for all 💪 |
Would you consider pointing to a Edit: I say this because the webpack loader and the Meteor build plugin both handle BTW, awesome plugin 😄 Edit2: Looks like it would only require a simple change here: https://github.com/apollographql/eslint-plugin-graphql/blob/master/src/index.js#L153-L154, adding a Edit3: We could use this code for it: https://github.com/nodkz/webpack-plugin-graphql-schema-hot/blob/master/index.js#L113-L124 |
@jamiter the best place to contribute to this discussion is here: kamilkisiela/graphql-config#20 This will soon be released and hopefully also finds its way back to the eslint plugin 🙂 |
This will be solved by #80 🙂 |
It would be a hassle to run a build script every time the schema changes, I'm sure there is a simpler way.
Blocks: https://github.com/apollostack/meteor-starter-kit/pull/5
The text was updated successfully, but these errors were encountered: