Skip to content
This repository was archived by the owner on Feb 12, 2024. It is now read-only.

Bitswap Quest#195

Merged
dignifiedquire merged 15 commits into
masterfrom
feat/bitswap
May 12, 2016
Merged

Bitswap Quest#195
dignifiedquire merged 15 commits into
masterfrom
feat/bitswap

Conversation

@dignifiedquire

@dignifiedquire dignifiedquire commented May 6, 2016

Copy link
Copy Markdown
Member

This is still a work in progress

TODO

  • Integrate bitswap
  • Simple block exchange test
  • Complex block exchange test
  • File exchange test
  • Bitswap Commands
    • HTTP
      • stat
      • wantlist
      • unwant will be done later
    • CLI
      • stat
      • wantlist
      • unwant will be done later
  • CI Green

Needs the following PRs

@jbenet jbenet added the status/in-progress In progress label May 6, 2016
@dignifiedquire dignifiedquire force-pushed the feat/bitswap branch 2 times, most recently from 2320cdf to 89de3fd Compare May 9, 2016 09:56
Comment thread .travis.yml
- npm run coverage-publish

env:
- CXX=g++-4.8

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why is this necessary?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

So that travis can build native modules (node 4 and greater require a c++ compiler for those) Details here: https://docs.travis-ci.com/user/languages/javascript-with-nodejs#Node.js-v4-(or-io.js-v3)-compiler-requirements

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I mean, which are the native modules that we have introduced with bitswap?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

None that we really need, travis was just complaining and I wanted to fix that (those native modules are mostly used through aegir)

@daviddias

Copy link
Copy Markdown
Member

LGTM :)

@dignifiedquire dignifiedquire merged commit c539741 into master May 12, 2016
@dignifiedquire dignifiedquire deleted the feat/bitswap branch May 12, 2016 11:30
@jbenet jbenet removed the status/in-progress In progress label May 12, 2016
@dignifiedquire

Copy link
Copy Markdown
Member Author

🎉 🎈 🎉 🎈 🎉 🎈 🎉 🎈 🎉 🎈 🎉 🎈 🎉 🎈 🎉 🎈

@daviddias

Copy link
Copy Markdown
Member

wowowowwowo :D

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