simple-navigation icon indicating copy to clipboard operation
simple-navigation copied to clipboard

make stack traces for errors in configs more usable

Open SKoschnicke opened this issue 7 years ago • 3 comments

When there is an error in the navigation config, the stack trace won't include a pointer to the actual config file and line but only something like

(eval):5:in `block in eval_config'

This change supplies the path to the config file to instance_eval so that the stack trace now include path and line where the error occurs.

SKoschnicke avatar Nov 07 '17 13:11 SKoschnicke

Coverage Status

Coverage decreased (-0.2%) to 99.446% when pulling 5c3590763796de524e444474c3e8641d2b502103 on SKoschnicke:fix_stack_traces into 3df3a88fd65645eba1d9fd8dcfd59804e9f6232c on codeplant:master.

coveralls avatar Nov 07 '17 13:11 coveralls

CI ftw! I will fix that...

SKoschnicke avatar Nov 07 '17 13:11 SKoschnicke

Okay, I need some help with that. Is it a good idea to just make the config_file method public?

SKoschnicke avatar Nov 07 '17 13:11 SKoschnicke