-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Fix broken exception recovery code when installing plugins #3487
Fix broken exception recovery code when installing plugins #3487
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good, I think.
The message: clearer, supports more than one spec
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the long delay getting into this!
Only minor comment would be to make the plugin names more clear in the error message. Maybe
Failed to install the following plugins: <comma_separated_list_with_names_wrapped_in_backquotes>. The underlying error was: <the_error_as_it_is_printed_now>
What do you think?
d23c7b7
to
4c82409
Compare
4c82409
to
c813260
Compare
@deivid-rodriguez Updated with your suggestion. |
Thanks so much, I'll merge this tomorrow! |
Fix broken exception recovery code when installing plugins (cherry picked from commit b277c8c)
Fix broken exception recovery code when installing plugins (cherry picked from commit b277c8c)
Fix broken exception recovery code when installing plugins (cherry picked from commit b277c8c)
Fix broken exception recovery code when installing plugins (cherry picked from commit b277c8c)
Fix broken exception recovery code when installing plugins (cherry picked from commit b277c8c)
Fix broken exception recovery code when installing plugins (cherry picked from commit b277c8c)
Fixes #3484