Skip to content
This repository has been archived by the owner on Mar 23, 2023. It is now read-only.

feat: add basic error codes and update test #8

Merged
merged 6 commits into from
Oct 1, 2018
Merged

Conversation

jacobheun
Copy link
Contributor

@jacobheun jacobheun commented Sep 12, 2018

Depends on ipfs/interface-datastore#22

Related to ipfs/js-ipfs#1557

This PR adds consistent error codes from the interface-datastore to allow downstream code to consistently handle datastore errors.

  • Update deps once interface-datastore#22 has been released

License: MIT
Signed-off-by: Jacob Heun <jacobheun@gmail.com>
@ghost ghost assigned jacobheun Sep 12, 2018
@ghost ghost added the status/in-progress In progress label Sep 12, 2018
@jacobheun jacobheun changed the title feat: add base error codes and update test feat: add basic error codes and update test Sep 12, 2018
License: MIT
Signed-off-by: Jacob Heun <jacobheun@gmail.com>
License: MIT
Signed-off-by: Jacob Heun <jacobheun@gmail.com>
License: MIT
Signed-off-by: Jacob Heun <jacobheun@gmail.com>
License: MIT
Signed-off-by: Jacob Heun <jacobheun@gmail.com>
Copy link
Member

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

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

Other than removing the console.log, everything else looks good!

examples/full-s3-repo/index.js Show resolved Hide resolved
@jacobheun
Copy link
Contributor Author

Thanks @vasco-santos, I will wait to merge this after the js-ipfs updates are released as the example will fail until that's done.

License: MIT
Signed-off-by: Jacob Heun <jacobheun@gmail.com>
@jacobheun jacobheun merged commit 31ba28a into master Oct 1, 2018
@ghost ghost removed the status/in-progress In progress label Oct 1, 2018
@jacobheun jacobheun deleted the feat/err-codes branch October 1, 2018 17:44
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.

2 participants