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

Add resend integration test and fix existing tests/comments #769

Merged
merged 2 commits into from
May 29, 2019

Conversation

chris-j-h
Copy link
Collaborator

@chris-j-h chris-j-h commented May 29, 2019

Related issue: #770

Related PR: #766

@codecov-io
Copy link

codecov-io commented May 29, 2019

Codecov Report

Merging #769 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #769   +/-   ##
=========================================
  Coverage     99.43%   99.43%           
  Complexity     1915     1915           
=========================================
  Files           288      288           
  Lines          5689     5689           
  Branches        288      288           
=========================================
  Hits           5657     5657           
  Misses            5        5           
  Partials         27       27

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 92dfb37...de04cd6. Read the comment docs.

@Krish1979 Krish1979 merged commit 70b9a93 into master May 29, 2019
@prd-fox prd-fox added 0.10.0 bug Something isn't working labels May 30, 2019
@chris-j-h chris-j-h deleted the resend-integration-tests branch May 30, 2019 13:02
@prd-fox prd-fox added the 0.9.2 label Jul 26, 2019
@prd-fox prd-fox added testing and removed 0.10.0 labels Aug 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.9.2 bug Something isn't working testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants