search

ianhattendorf / autocomplete-ruby

Provides intelligent code completion for Ruby in the Atom editor. Requires RSense.
MIT License
51 stars 9 forks source link

Error origin #15

Closed jneuendorf closed 7 years ago

jneuendorf commented 7 years ago

Hi,

I recently often encountered the error (displayed as a notification) that's generated here. I had to search the internet to find out that this is related to this atom package.

I think it would be nice to make it easier to find the package related to an error. Therefore I thought about creating a PR but wanted to check for the best solution first. Two simple solutions I can think of are:

Which one to you prefer? Or do you have a better/different solution?

ianhattendorf commented 7 years ago

Sure, feel free to prepend the package name to the error messages. Is there a common format for this used in other packages? i.e. autocomplete-ruby: exec error: ...

console.error sounds like it would be helpful as well, as long as this isn't discouraged in electron apps.

jneuendorf commented 7 years ago

As to http://stackoverflow.com/a/6856325 console logging should not be a great concern. I am not aware of a common naming format but I guess prepending the exact package name would definitely be a good start. ;)

Therefore, I would then create a PR soon :)