Skip to content
This repository has been archived by the owner on Aug 11, 2021. It is now read-only.

fix: remove node globals #124

Merged
merged 2 commits into from
Mar 27, 2020
Merged

fix: remove node globals #124

merged 2 commits into from
Mar 27, 2020

Conversation

hugomrdias
Copy link
Member

add buffer
update borc

add buffer
update borc
@codecov
Copy link

codecov bot commented Mar 25, 2020

Codecov Report

Merging #124 into master will increase coverage by 0.07%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
+ Coverage   96.11%   96.19%   +0.07%     
==========================================
  Files           3        3              
  Lines         103      105       +2     
==========================================
+ Hits           99      101       +2     
  Misses          4        4              
Impacted Files Coverage Δ
src/resolver.js 95.83% <100.00%> (+0.18%) ⬆️
src/util.js 96.10% <100.00%> (+0.05%) ⬆️

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 bc6ac54...de2bce3. Read the comment docs.

package.json Outdated
@@ -37,7 +37,8 @@
},
"homepage": "https://github.com/ipld/js-ipld-dag-cbor",
"dependencies": {
"borc": "^2.1.0",
"borc": "hugomrdias/borc#fix/remove-node-globals",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should get a new borc release out soon, so this probably isn't ideal

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes ofc forgot to mention it in the op

@hugomrdias hugomrdias requested review from vmx and rvagg March 26, 2020 19:33
@vmx vmx merged commit 7acfeae into master Mar 27, 2020
@vmx vmx deleted the fix/remove-node-globals branch March 27, 2020 10:52
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.

3 participants