Jed icon indicating copy to clipboard operation
Jed copied to clipboard

Add meaningful error messages

Open atimmer opened this issue 8 years ago • 4 comments

I was bitten by [sprintf] huh?, this should make debugging for other people much easier.

atimmer avatar May 09 '16 20:05 atimmer

Another possibility would be depending on https://github.com/alexei/sprintf.js which looks at lot like the inline sprintf-implementation.

atimmer avatar May 09 '16 20:05 atimmer

It is the alexei implementation, just inlined. Feel free to just re-inline the latest.

SlexAxton avatar May 09 '16 22:05 SlexAxton

Wouldn't it be better to depend on it in the package.json? This means updates in the future will be easier to incorporate. I am willing to do this work.

atimmer avatar May 13 '16 18:05 atimmer

Happy to take a PR. It works like it currently does because I released Jed in 2012 and sprintf wasn't on Github until 2013. It was just a file on his website.

SlexAxton avatar May 13 '16 18:05 SlexAxton