Skip to content

bump graphiql to 0.13.0, and rename __debug to _debug#646

Merged
phalt merged 1 commit intographql-python:masterfrom
dulmandakh:graphiql-0.13.0
May 20, 2019
Merged

bump graphiql to 0.13.0, and rename __debug to _debug#646
phalt merged 1 commit intographql-python:masterfrom
dulmandakh:graphiql-0.13.0

Conversation

@dulmandakh
Copy link
Copy Markdown
Contributor

This PR bumps graphiql to 0.13.0, the latest version available on JS CDN, and includes many improvements. And since 0.12 GraphQL prohibits to use names that starts with __, so renamed __debug to _debug. It works for me.

I use graphene-django from master.

fixes #640

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 92.712% when pulling 49aedf1 on dulmandakh:graphiql-0.13.0 into 74263b2 on graphql-python:master.

4 similar comments
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 92.712% when pulling 49aedf1 on dulmandakh:graphiql-0.13.0 into 74263b2 on graphql-python:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 92.712% when pulling 49aedf1 on dulmandakh:graphiql-0.13.0 into 74263b2 on graphql-python:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 92.712% when pulling 49aedf1 on dulmandakh:graphiql-0.13.0 into 74263b2 on graphql-python:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 92.712% when pulling 49aedf1 on dulmandakh:graphiql-0.13.0 into 74263b2 on graphql-python:master.

Copy link
Copy Markdown
Contributor

@phalt phalt left a comment

Choose a reason for hiding this comment

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

This looks good to me, as you're using this project from master you can attest to it working?

@dulmandakh
Copy link
Copy Markdown
Contributor Author

dulmandakh commented May 20, 2019

@phalt yep, it works for me. I'll open issues or create PRs if I encounter issues.

@phalt
Copy link
Copy Markdown
Contributor

phalt commented May 20, 2019

@dulmandakh thanks for this. I'll ear-mark it for being part of the 2.3 release (still getting the Graphene team to organise how to do this easier).

@phalt phalt merged commit 1d1b9e8 into graphql-python:master May 20, 2019
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.

Name "__debug" must not begin with "__", In a future release of graphql this will become a hard error.

3 participants