GET as a transport option instead of a parameter + ErrorCancellable #599

Merged

designatednerd merged 18 commits intomasterfrom

update/get-as-option

Jul 2, 2019

Commits on Jul 2, 2019

Add `ErrorCancellable` types so we can bail out of things which still need `Cancelable` after a completion handler has been called.
designatednerd
committed
update to 2-space indentation on all projects by default
designatednerd
committed
Make error with no additional parameters just an enum
designatednerd
committed
tweak a few descriptions for more consistent swifting
designatednerd
committed
pull request and response errors out into their own files
designatednerd
committed
make error cancelable final
designatednerd
committed
enable code coverage for main, SQLite, and WebSocket schemes
designatednerd
committed
add public initializer and fix indentation for ErrorCancellable
designatednerd
committed
fix swift warnings in the WebSocket tests
designatednerd
committed
Make GET for queries an option on transport rather passing parameter everywhere; FetchHTTPMethod -> GraphQLHTTPMethod
designatednerd
committed
HTTP method no longer needs to be public
designatednerd
committed
rm whitespace
designatednerd
committed
Separator and explanation of why the init is public
designatednerd
committed
remove tests that were duplicated to test `GET`
designatednerd
committed
rename ErrorCancellable -> EmptyCancellable for flexibility
designatednerd
committed
Fix a couple `ErrorCancellable`s the renamer missed 🤦‍♀️
designatednerd
committed
Merge branch 'master' into update/get-as-option + fix merge conflicts
designatednerd
committed
fix indentation in web socket transport
designatednerd
committed

⚡