-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 descriptions missing in tree of plugin #10253
fix descriptions missing in tree of plugin #10253
Conversation
I confirm that this allows descriptions in tree views to show. I also confirm that the |
I think the behaviour was introduced within 27c0050, so it would be helpful if @vitaliy-guliy could take a look @vitaliy-guliy |
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.
The changes are reasonable and fix the stated problem. Unless @vitaliy-guliy objects, I'll plan to merge at the end of the week.
@shuyaqian, would you mind adding a brief breaking change notice in the changelog about removing the |
Signed-off-by: shuyaqian [email protected]
Done. |
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.
@shuyaqian
thanks for the fix!
@colin-grant-work @vince-fugnitto
thank you very much for the heads-up and time to check if the changes don't break our assembly!
Signed-off-by: shuyaqian [email protected]
What it does
getCaption
How to test
in vscode:
Review checklist
Reminder for reviewers