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

Fix multiple import declarations from the same path not working #140

Merged
merged 1 commit into from
Apr 26, 2018

Conversation

SpaceK33z
Copy link
Collaborator

@SpaceK33z SpaceK33z commented Apr 18, 2018

For more info see #115. The TL;DR is that currently it is not possible to do something like this:

# import Query.first, Query.second from "a.graphql"
# import Query.third from "a.graphql"

I dove into the source code and found out this bug occurs because of the way circular dependencies are handled. I have no idea if this is the best way to fix this issue, so I'm open to better ways.

@schickling
Copy link
Contributor

🎉 This PR is included in version 0.5.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@SpaceK33z SpaceK33z deleted the fix-duplicate-imports branch April 26, 2018 18:46
@SpaceK33z
Copy link
Collaborator Author

That's awesome, thank you!

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.

None yet

2 participants