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
print_schema no more put a trailing new line #157
Comments
Perfect!
You do what you have time to do, and your work is already marvellous. The fault is on my side, as I should have look at GraphQL.js changelog, as it was finally simple to find in it. |
* Bump graphql-core from 3.1.7 to 3.2.0 in /tests Bumps [graphql-core](https://github.com/graphql-python/graphql-core) from 3.1.7 to 3.2.0. - [Release notes](https://github.com/graphql-python/graphql-core/releases) - [Commits](graphql-python/graphql-core@v3.1.7...v3.2.0) --- updated-dependencies: - dependency-name: graphql-core dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * Fix trailing new lines in graphql schema See graphql-python/graphql-core#157 Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Joseph Perez <joperez@hotmail.fr>
* Bump graphql-core from 3.1.7 to 3.2.0 in /tests Bumps [graphql-core](https://github.com/graphql-python/graphql-core) from 3.1.7 to 3.2.0. - [Release notes](https://github.com/graphql-python/graphql-core/releases) - [Commits](graphql-python/graphql-core@v3.1.7...v3.2.0) --- updated-dependencies: - dependency-name: graphql-core dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * Fix trailing new lines in graphql schema See graphql-python/graphql-core#157 Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Joseph Perez <joperez@hotmail.fr>
Reporting issues with GraphQL-core 3
Release of version 3.2.0 triggered a dependabot PR on my repository, but all my tests are failing because
print_schema
behavior has changed and doesn't print a new line at the end of the printed schema. See yourself here.My library can indeed be used to generate GraphQL schema, so most of my GraphQL related tests are making assertion on the printed schema; it's maybe not the best idea but I found it to be the simplest. For example, this test is now failing because of the missing trailing newline:
I didn't found the reason of this change in the changelog and in a quick blame of print_schema.py, so I would like to simply ask you if this behavior is the expected one, as maybe GraphQL.js has changed its behavior too (I did not look at the detail of their changelog).
If it's indeed a regression, I'm ok to try to resolve it and submit a PR. Otherwise, I will just update all my tests, that's not a problem at all.
The text was updated successfully, but these errors were encountered: