Skip to content
This repository has been archived by the owner on May 7, 2023. It is now read-only.

Remove apollo client #135

Merged
merged 2 commits into from
Jun 9, 2020
Merged

Remove apollo client #135

merged 2 commits into from
Jun 9, 2020

Conversation

hyochan
Copy link
Owner

@hyochan hyochan commented Jun 8, 2020

Description

We'd like to migrate our project from Apollo client to relay.

Related Issues

#134

Tests

Removed all the MockedProvider.

Checklist

Before you create this PR confirms that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I signed the [CLA].
  • Run yarn lint && yarn tsc
  • Run yarn test or yarn test -u if you need to update snapshot.
  • I am willing to follow-up on review comments in a timely manner.

@codecov
Copy link

codecov bot commented Jun 8, 2020

Codecov Report

Merging #135 into master will decrease coverage by 0.16%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #135      +/-   ##
==========================================
- Coverage   81.51%   81.35%   -0.17%     
==========================================
  Files          37       37              
  Lines        1228     1137      -91     
  Branches      131      113      -18     
==========================================
- Hits         1001      925      -76     
+ Misses        165      155      -10     
+ Partials       62       57       -5     

@hyochan hyochan merged commit 8800b53 into master Jun 9, 2020
@hyochan hyochan deleted the migration/uninstall-apollo branch June 9, 2020 00:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant