Document, test, and improve functionality. #46
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
License: MIT Signed-off-by: Luke Stack <stackjl@appstate.edu>
License: MIT Signed-off-by: Zlati Gardev zgardev@yahoo.com
…t_http.py. License: MIT Signed-off-by: Ryan Soley <resoley@us.ibm.com>
Most of the docs were based on or copied from the IPFS Apiary and/or the CLI itself. Much will need to be updated as the project progresses. License: MIT Signed-off-by: Luke Stack <stackjl@appstate.edu>
I also removed the pep8 checks for __init__.py files since necessary 'unused imports' often appear. License: MIT Signed-off-by: Luke Stack <stackjl@appstate.edu>
License: MIT Signed-off-by: Christopher Waldon <christopher.waldon.dev@gmail.com>
License: MIT Signed-off-by: Christopher Waldon <christopher.waldon.dev@gmail.com>
Add functionality fixed and correctly tested. Object path subcommands scaffolded. Object new command implemented and tested. Block get, block stat, and block put tested. License: MIT Signed-off-by: Christopher Waldon <christopher.waldon.dev@gmail.com>
License: MIT Signed-off-by: Ryan Soley <resoley@us.ibm.com>
License: MIT Signed-off-by: Ryan Soley <resoley@us.ibm.com>
License: MIT Signed-off-by: Christopher Waldon <christopher.waldon.dev@gmail.com>
License: MIT Signed-off-by: Christopher Waldon <christopher.waldon.dev@gmail.com>
License: MIT Signed-off-by: Christopher Waldon <christopher.waldon.dev@gmail.com>
|
Actually, we're not quite ready yet. Just realized this branch is missing something. Will open a new one when fixed. |
Added pin tests Added log functions/tests Added other random tests Removed tailing separator on directory paths in multipart License: MIT Signed-off-by: Luke Stack <stackjl@appstate.edu>
|
Okay, fixed now.
Additionally, we've fixed and tested the Files extension and the Log extension:
|
This was referenced Jul 23, 2016
Closed
|
This is great, thanks so much! :) I have only very little experience with python so I can't give much valuable feedback. @Alexander255 @amstocker @jbenet what do you think? |
|
👍 on this, we're currently shelling out to go-ipfs to do recursive adds |
This was referenced Aug 10, 2016
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We've written unit and functional tests for most of the project. Additionally, we've implemented some chunks of the core spec that were missing. We have all of the Core API spec implemented with the exception of node_start, node_stop, repo_init, repo_config_get, and repo_config_set. We also have a working version of add, cat, and get. (Add does not currently handle filtering added files by a regular expression though.)
Resolves #45 and #34 .