Skip to content

Clear old data on fetch #206

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

Merged
merged 1 commit into from
Feb 12, 2016
Merged

Clear old data on fetch #206

merged 1 commit into from
Feb 12, 2016

Conversation

andrewimm
Copy link
Contributor

Handles Object.fetch() and Query results

@codecov-io
Copy link

Current coverage is 79.58%

Merging #206 into master will increase coverage by +0.14% as of ea98a49

@@            master    #206   diff @@
======================================
  Files           39      39       
  Stmts         3007    3012     +5
  Branches       719     721     +2
  Methods          0       0       
======================================
+ Hit           2389    2397     +8
- Partial        202     207     +5
+ Missed         416     408     -8

Review entire Coverage Diff as of ea98a49

Powered by Codecov. Updated on successful CI builds.

@drew-gross
Copy link

Seems to make sense. Integration tests sounds like a good idea, feel free to merge this before or after making those.

@andrewimm
Copy link
Contributor Author

Integration tests run against the latest "master," so I'll merge now.

andrewimm added a commit that referenced this pull request Feb 12, 2016
@andrewimm andrewimm merged commit 8f55741 into master Feb 12, 2016
@dplewis dplewis deleted the clear_on_fetch branch March 21, 2019 14:34
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.

4 participants