zino-hofmann / graphql-flutter

A GraphQL client for Flutter, bringing all the features from a modern GraphQL client to one easy to use package.
https://zino-hofmann.github.io/graphql-flutter
MIT License
3.25k stars 620 forks source link

prepare version v5.1.2-beta.1 of graphql client #1132

Closed vincenzopalazzo closed 2 years ago

vincenzopalazzo commented 2 years ago

Let's put alive the new protocol support, we can find some bugs that we can find during the refactoring period!

Signed-off-by: Vincenzo Palazzo vincenzopalazzodev@gmail.com

codecov[bot] commented 2 years ago

Codecov Report

Merging #1132 (0edb38d) into main (ad8aec7) will decrease coverage by 2.70%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1132      +/-   ##
==========================================
- Coverage   57.53%   54.83%   -2.71%     
==========================================
  Files          41       13      -28     
  Lines        1658      248    -1410     
==========================================
- Hits          954      136     -818     
+ Misses        704      112     -592     
Impacted Files Coverage Δ
packages/graphql/lib/src/cache/cache.dart
packages/graphql/lib/src/utilities/helpers.dart
...lib/src/links/websocket_link/websocket_client.dart
packages/graphql/lib/src/core/query_options.dart
...ackages/graphql/lib/src/exceptions/exceptions.dart
...ackages/graphql/lib/src/exceptions/network_io.dart
...raphql/lib/src/cache/_optimistic_transactions.dart
packages/graphql/lib/src/core/query_manager.dart
...b/src/links/websocket_link/websocket_messages.dart
...es/graphql/lib/src/core/_query_write_handling.dart
... and 18 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 ad8aec7...0edb38d. Read the comment docs.

vincenzopalazzo commented 2 years ago

This PR required this change https://github.com/zino-hofmann/graphql-flutter/pull/1133 before merging, or also after :) but it requires this!