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

fix: return operation array from getOperation method #816

Merged
merged 2 commits into from
Apr 24, 2020

Conversation

xiaozhenliu-gg5
Copy link
Contributor

@xiaozhenliu-gg5 xiaozhenliu-gg5 commented Apr 24, 2020

@codecov
Copy link

codecov bot commented Apr 24, 2020

Codecov Report

Merging #816 into master will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #816      +/-   ##
==========================================
+ Coverage   89.95%   90.00%   +0.04%     
==========================================
  Files          48       48              
  Lines        8133     8133              
  Branches      479      480       +1     
==========================================
+ Hits         7316     7320       +4     
+ Misses        814      810       -4     
  Partials        3        3              
Impacted Files Coverage Δ
src/operationsClient.ts 89.73% <100.00%> (ø)
.mocharc.js 78.57% <0.00%> (-14.29%) ⬇️
src/normalCalls/retries.ts 98.71% <0.00%> (+5.12%) ⬆️

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 f3a7e4b...e85d7eb. Read the comment docs.

@xiaozhenliu-gg5 xiaozhenliu-gg5 added the cla: yes This human has signed the Contributor License Agreement. label Apr 24, 2020
@xiaozhenliu-gg5 xiaozhenliu-gg5 merged commit 483ae8c into master Apr 24, 2020
@xiaozhenliu-gg5 xiaozhenliu-gg5 deleted the return-operation-array branch April 24, 2020 21:00
@alexander-fenster alexander-fenster added release-please:force-run To run release-please and removed release-please:force-run To run release-please labels Apr 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants