Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reverse stacktraces (again) #114

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

reverse stacktraces (again) #114

wants to merge 1 commit into from

Conversation

pfitzseb
Copy link
Member

Fixes JunoLab/atom-julia-client#374.

I'm strongly in favour of this just to be consistent -- is this okay with you, @MikeInnes?

@MikeInnes
Copy link
Member

Well we could also be consistent, at least within Juno, by changing the REPL stack trace order.

I can go along with this if there's a general consensus that consistency is more important here, but I really do think it's the wrong default for 99% of Julia users (especially the ones using Juno), who usually have an issue in their code, not at the deepest level of the base system.

@aviatesk
Copy link
Member

aviatesk commented Aug 5, 2019

I'm actually very in favour of @pfitzseb .
I'm just an inexperienced Julia programmer, but feel like we beginners want to first refer to the outermost call and then continue to dig into inner calls since it would be easier to start thinking from higher level of the error (assuming people usually look at errors from up to down).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stack trace reversed in juno
3 participants