-
Notifications
You must be signed in to change notification settings - Fork 110
Graphql dependency error #191
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
Graphql dependency error #191
Conversation
Also resolves #187 as well. |
cc @abawchen |
@abawchen Any way we can merge and release this? |
Hi, I also have an issue where I can't install |
fyi graphene 3.1 solves quite a bit of dephell that's still existing in 3.0 ref ciscorn/starlette-graphene3#26 (comment) in comparison: $ pipgrip --tree graphene==3.1 graphql-core==3.2.*
graphene==3.1 (3.1)
├── aniso8601<10,>=8 (9.0.1)
├── graphql-core<3.3,>=3.1 (3.2.1)
└── graphql-relay<3.3,>=3.1 (3.2.0)
└── graphql-core<3.3,>=3.2 (3.2.1)
graphql-core==3.2.* (3.2.1)
$ pipgrip --tree graphene==3.1 graphql-core==3.1.*
graphene==3.1 (3.1)
├── aniso8601<10,>=8 (9.0.1)
├── graphql-core<3.3,>=3.1 (3.1.7)
└── graphql-relay<3.3,>=3.1 (3.1.5)
└── graphql-core<3.2,>=3.1 (3.1.7)
graphql-core==3.1.* (3.1.7)
$ pipgrip --tree graphene==3.0
graphene==3.0 (3.0)
├── aniso8601<10,>=8 (9.0.1)
├── graphql-core~=3.1.2 (3.1.7)
└── graphql-relay<4,>=3.0 (3.1.5)
└── graphql-core<3.2,>=3.1 (3.1.7) |
Have you run the tests with graphene 3.1? |
I have run the test suite
|
With graphene==3.0
|
@blochs : Now you have the permission to merge, free to go. Thanks. |
for stability as @leonardwellthy mentions, and to increase cross-compatibility as I mentioned, could this PR bump graphene to 3.1 instead of to 3.0? |
Resolves #189 with the introduction of the graphene v3.0 package as well.
All tests pass.