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

Wired up the trickle dag flag for the add command #1379

Merged
merged 1 commit into from
Jun 17, 2015

Conversation

dylanPowers
Copy link
Member

Fixes #1368

@GitCop
Copy link

GitCop commented Jun 16, 2015

There were the following issues with your Pull Request

  • Commit: dffecee
    • Invalid signoff. Commit message must end with License: MIT
      Signed-off-by: .* <.*>

Guidelines are available to help. Your feedback on GitCop is welcome on this issue


This message was auto-generated by https://gitcop.com

License: MIT
Signed-off-by: Dylan Powers <dylan.kyle.powers@gmail.com>
@dylanPowers
Copy link
Member Author

Added the signed-off-by lines

@whyrusleeping
Copy link
Member

this LGTM

whyrusleeping added a commit that referenced this pull request Jun 17, 2015
Wired up the trickle dag flag for the add command
@whyrusleeping whyrusleeping merged commit b46712b into ipfs:master Jun 17, 2015
@jbenet
Copy link
Member

jbenet commented Jun 17, 2015

This needs a test case. We had this in there for weeks without knowing it didn't work... @whyrusleeping please don't merge things without tests

@dylanPowers
Copy link
Member Author

I'm writing the test right now

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.

4 participants