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

Master to async await #493

Merged
merged 17 commits into from
Jan 19, 2020
Merged

Master to async await #493

merged 17 commits into from
Jan 19, 2020

Conversation

iancormac84
Copy link
Contributor

Attempt at rebase of async-await branch on to master. One test fails.

klausi and others added 17 commits December 26, 2019 04:48
The `uuid` maintainers have started releasing in the `0.8` version train.  I've relaxed the version requirements on `juniper`'s dependencies to allow juniper users to specify a different version in their `Cargo.toml` and still have the integration work.
incl. upgrade to latest React
Was copied from undefined variable error message.
* Validate variables of the executed operation only

* Use `unreachable!` in `validate_var_defs`.

Use `unreachable!` instead of `panic!` on invalid variable types,
since thay have already been checked during document validation.

* Fix formatting in `validation/input_value.rs`
@codecov-io
Copy link

codecov-io commented Jan 12, 2020

Codecov Report

Merging #493 into async-await will decrease coverage by 0.16%.
The diff coverage is 91.3%.

Impacted file tree graph

@@               Coverage Diff               @@
##           async-await     #493      +/-   ##
===============================================
- Coverage        84.53%   84.37%   -0.17%     
===============================================
  Files              112      112              
  Lines            15966    16076     +110     
===============================================
+ Hits             13497    13564      +67     
- Misses            2469     2512      +43
Impacted Files Coverage Δ
juniper_codegen/src/lib.rs 0% <ø> (ø) ⬆️
juniper/src/value/mod.rs 85.58% <ø> (ø) ⬆️
juniper/src/http/playground.rs 0% <ø> (ø) ⬆️
juniper/src/executor_tests/variables.rs 96.19% <ø> (+0.17%) ⬆️
juniper/src/http/graphiql.rs 0% <ø> (ø) ⬆️
...uniper/src/validation/rules/no_unused_variables.rs 93.1% <100%> (+0.04%) ⬆️
juniper/src/lib.rs 84.37% <100%> (+0.5%) ⬆️
juniper/src/executor_tests/executor.rs 99.77% <100%> (ø) ⬆️
juniper/src/validation/input_value.rs 83.5% <83.33%> (-1.28%) ⬇️
juniper/src/executor/mod.rs 91.64% <92%> (+0.1%) ⬆️
... and 44 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c2f0dd2...11c411b. Read the comment docs.

@LegNeato LegNeato merged commit 001cabc into graphql-rust:async-await Jan 19, 2020
@LegNeato
Copy link
Member

Wow, thank you! I haven't had a ton of time due to work and this should help unblock a ton of folks.

@iancormac84 iancormac84 deleted the master-to-async-await branch January 19, 2020 02:19
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.

8 participants