restore Error: prefix

[Stebalien]

this is a breaking change and many of our tests expect this

[codecov]

Codecov Report

Merging #81 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master      #81   +/-   ##
=======================================
  Coverage   43.02%   43.02%           
=======================================
  Files          23       23           
  Lines        1922     1922           
=======================================
  Hits          827      827           
  Misses        960      960           
  Partials      135      135

Impacted Files	Coverage Δ
cli/run.go	0% <0%> (ø)	⬆️

---

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 3940b10...b32936a. Read the comment docs.

[dignifiedquire]

any chance we can make that an option? otherwise I will need to maintain a copy with that change :/

[Stebalien]

any chance we can make that an option? otherwise I will need to maintain a copy with that change

I'd rather not; I can't see a clean way of doing that without adding yet another argument to that function. Why do you need this change?

Ideally, we wouldn't print any errors, we'd just return them. However, that makes printing help awkward.

[Stebalien]

I'm going to merge this for now as this was a breaking change, we need to cut a go-ipfs release, and I didn't really consider the impact when I merged #68. We can discuss solutions to this in #82.