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

[release/6.0] [wasm][debugger] View object attributes - DebugType=full #62543

Merged
merged 1 commit into from
Dec 15, 2021

Conversation

thaystg
Copy link
Member

@thaystg thaystg commented Dec 8, 2021

Backport of #62278 to release/6.0

/cc @thaystg

Customer Impact

Show object attributes even if this object is an instantiation of a class that is in a library that uses DebugType=full. On .net 5 this works, so it's a regression.

Testing

Unit tests and manual tests on Blazor.

Risk

Low Risk, only adding a try catch when trying to read the PDB.

@thaystg thaystg added Servicing-consider Issue for next servicing release review area-Debugger-mono labels Dec 8, 2021
@thaystg thaystg requested a review from lewing December 8, 2021 17:50
@thaystg thaystg requested a review from marek-safar as a code owner December 8, 2021 17:50
@ghost
Copy link

ghost commented Dec 8, 2021

Tagging subscribers to this area: @thaystg
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #62278 to release/6.0

/cc @thaystg

Customer Impact

Show object attributes even if this object is an instantiation of a class that is in a library that uses DebugType=full. On .net 5 this works, so it's a regression.

Testing

Unit tests and manual tests on Blazor.

Risk

Low Risk, only adding a try catch when trying to read the PDB.

Author: thaystg
Assignees: -
Labels:

Servicing-consider, area-Debugger-mono

Milestone: -

@thaystg
Copy link
Member Author

thaystg commented Dec 8, 2021

@lewing I did a partial backport of the PR to avoid unnecessary improvements that could cause any side effect.

@thaystg thaystg added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Dec 15, 2021
@thaystg
Copy link
Member Author

thaystg commented Dec 15, 2021

Approved by email.

@safern
Copy link
Member

safern commented Dec 15, 2021

We can merge whenever a reviewer approves the PR.

@thaystg thaystg requested a review from radical December 15, 2021 19:01
@safern safern merged commit 2e0a367 into dotnet:release/6.0 Dec 15, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jan 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Debugger-mono Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants