Skip to content

Allow fragments only for codegen#707

Merged
trevor-scheer merged 2 commits intomasterfrom
trevor/support-frags-only
Nov 15, 2018
Merged

Allow fragments only for codegen#707
trevor-scheer merged 2 commits intomasterfrom
trevor/support-frags-only

Conversation

@trevor-scheer
Copy link
Copy Markdown
Contributor

Only throw an error if we have neither operations or fragments.

Tested this on a demo project with fragments only to confirm the fix.

Fixes #684

Copy link
Copy Markdown
Contributor

@martijnwalraven martijnwalraven left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Comment thread packages/apollo/src/commands/client/codegen.ts Outdated
Only throw an error if we have neither operations _or_ fragments
@trevor-scheer trevor-scheer force-pushed the trevor/support-frags-only branch from 400a7fa to 311bda0 Compare November 15, 2018 17:06
@trevor-scheer trevor-scheer merged commit 08b3b84 into master Nov 15, 2018
@trevor-scheer trevor-scheer deleted the trevor/support-frags-only branch November 15, 2018 17:09
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.

Fragments-only set of files are not supported

2 participants