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.

Sign up for GitHub

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

Jump to bottom

#726: Mark field as resolver with graphql directives #727

Closed
wants to merge 1 commit into from
Closed

#726: Mark field as resolver with graphql directives #727

wants to merge 1 commit into from

Conversation

sunfmin
Copy link
Contributor

@sunfmin sunfmin commented May 24, 2019 •
edited
Loading

Describe your PR and link to any relevant issues.

I have:

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

@sunfmin
Copy link
Contributor Author

sunfmin commented May 24, 2019

I haven't been able to find existing tests that test generation of go code and seems all existing tests passed.

@lwc lwc mentioned this pull request May 30, 2019
Merged
2 tasks
@vektah
Copy link
Collaborator

vektah commented Jun 18, 2019

Closing for #732 which adds a few more directives

@vektah vektah closed this Jun 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sunfmin @vektah