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

Added start for webpack/babel compatible declarations #2732

Closed
wants to merge 2 commits into from
Closed

Added start for webpack/babel compatible declarations #2732

wants to merge 2 commits into from

Conversation

CarloPalinckx
Copy link
Contributor

@dotansimha @mxstbr I've started working on making a plugin able to output "declarations only". In order to be compatible with something like a babel-macro or webpack-loader.

A few things I would like some feedback on before continuing:

I think there are two approaches we can take. Either we make "declarations only" a configuration option per plugin. Or we can adjust the graphql-module plugin to somehow include the modified output of specific compatible plugins.

I could also use some pointers on how to get the document file path inside a plugin nicely, that way we can declare the module with the path accordingly.

Would love to hear your thoughts 🙌

@dotansimha
Copy link
Owner

Nice! Thank you @CarloPalinckx !

@kamilkisiela can you please take a look? you are the last one that tried to integrate Webpack with the Codegen ;)

@dotansimha dotansimha self-assigned this Feb 12, 2020
@dotansimha dotansimha closed this Mar 3, 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.

2 participants