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

Add stub command #323

Closed
wants to merge 2 commits into from
Closed

Add stub command #323

wants to merge 2 commits into from

Conversation

vektah
Copy link
Collaborator

@vektah vektah commented Aug 28, 2018

WIP - very incomplete

This adds a new gqlgen stubs command that compares the generated interfaces and creates stubs for any missing ones, updating and signatures along the way.

Because this is mutating existing user-written source code I want to keep it in a separate command for now. As we gain some confidence in it I imagine it would happen automatically in gqlgen generate if configured.

Fixes #216

I have:

  • Added tests covering the bug / feature (see testing)
  • Updated any relevant documentation (see docs)

@vektah vektah mentioned this pull request Sep 20, 2018
@mathewbyrne
Copy link
Contributor

Closing for now, some of this might be useful later.

@frederikhors
Copy link
Collaborator

@mathewbyrne why later and not now?

@mathewbyrne
Copy link
Contributor

why later and not now?

I'm not sure what you mean. This PR itself will never merge, but some sections of it may be useful when we do address stub generation at some point. We are currently addressing some more fundamental refactors to code generation and type mapping that are more of a priority at the moment.

@porty porty deleted the gqlgen-sync branch February 8, 2019 18:37
@vektah vektah mentioned this pull request Jan 20, 2020
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