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

run book tests and fix book examples #940

Merged
merged 3 commits into from Jun 4, 2022

Conversation

urkle
Copy link
Contributor

@urkle urkle commented Jun 2, 2022

run mdbook tests so we ensure the examples actually compile

There are several examples that were broken and did not actually work (thus making it hard to learn to use async-graphql)

urkle added 2 commits June 2, 2022 17:27
- examples to fix still
  - error_extensions.md ResultExt example does not compile!
     - trait ErrorExtensions is not implemented for ParseIntError
  - dataloader
     - requires sqlx to work. So we either "stub" it OR we rewrite them simpler to use a  simple "faux" db library
@sunli829
Copy link
Collaborator

sunli829 commented Jun 3, 2022

awesome, thank you so much for making this! 👏

…rust version

- this has to exclude integrations/tide as an indirect dependency requires rust 1.59.0
@sunli829 sunli829 merged commit 713938c into async-graphql:master Jun 4, 2022
@urkle urkle deleted the feat-book-tests branch June 6, 2022 17:00
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