-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Update custom errors with Exception#full_message #8488
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
base: master
Are you sure you want to change the base?
Conversation
|
@deivid-rodriguez , now I wondering whether I should get rid of the "Backtrace" part of the message on L140-124 as well as on L#213-215 ? Thanks for reviewing when you'll be ready. |
|
@NeimadTL Can you show how these errors look now? It seems to me that they are going to start showing redundant information now, because So, yeah, regarding your question, yes, I think |
|
Hey @deivid-rodriguez, thanks for your feedback. According to me (but please correct me if I'm wrong), the ran spec seems to be the correct one to see the message but the thing is, even by changing the the |
|
What debug print are you adding so that running the spec prints that output? My suggestion was more to try a realworld case. To do this, you can do the following. Create a sample app and source "https://rubygems.org"
gem "rake"Configure it to install to a local path, for example, The installed Now edit that file to force a Now you should be able to see the new error by running: And you can compare it with the error that your globally installed bundler shows: |
|
Hi @deivid-rodriguez, thanks for your help. Can you explain to me how to do that part please:
|
The use of `Exception#full_message` makes more sense as it shows the cause and the backstrace.
0efeaa5 to
62a92c3
Compare

What was the end-user or developer problem that led to this PR?
Not an issue per se but more of an enhancement related to issue #8067
What is your fix for the problem, implemented in this PR?
Make use of
Exception#full_messageinstead ofException#messageso the cause and the backstrace show up.Make sure the following tasks are checked