Skip to content

fix: use the correct API version header#60

Merged
faustbrian merged 4 commits intomasterfrom
faustbrian-patch-1
Jan 15, 2019
Merged

fix: use the correct API version header#60
faustbrian merged 4 commits intomasterfrom
faustbrian-patch-1

Conversation

@faustbrian
Copy link
Copy Markdown
Contributor

Proposed changes

ArkEcosystem/core#1970

Types of changes

  • Bugfix (non-breaking change which fixes an issue)

Checklist

  • I have read the CONTRIBUTING documentation
  • Lint and unit tests pass locally with my changes

@faustbrian faustbrian added the Status: Blocked The pull request is blocked from being merged for the time being. label Jan 15, 2019
@faustbrian faustbrian changed the title fix: use the correct API vendor fix: use the correct API version header Jan 15, 2019
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #60 into master will increase coverage by 0.41%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
+ Coverage   73.42%   73.84%   +0.41%     
==========================================
  Files          22       22              
  Lines         301      302       +1     
==========================================
+ Hits          221      223       +2     
+ Misses         80       79       -1
Impacted Files Coverage Δ
src/http/os/http.cpp 60.71% <ø> (+5.15%) ⬆️

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 be8db44...561754f. Read the comment docs.

@faustbrian faustbrian removed the Status: Blocked The pull request is blocked from being merged for the time being. label Jan 15, 2019
@faustbrian faustbrian merged commit e2325ba into master Jan 15, 2019
@faustbrian faustbrian deleted the faustbrian-patch-1 branch January 15, 2019 09:39
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.

2 participants