Skip to content

[release/7.0][wasm][debugger] Indexing with expression #75559

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

Merged
merged 3 commits into from
Sep 14, 2022

Conversation

github-actions[bot]
Copy link
Contributor

Backport of #75524 to release/7.0

/cc @ilonatommy

Customer Impact

Testing

Risk

IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.

@ghost
Copy link

ghost commented Sep 13, 2022

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

Issue Details

Backport of #75524 to release/7.0

/cc @ilonatommy

Customer Impact

Testing

Risk

IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.

Author: github-actions[bot]
Assignees: -
Labels:

area-Debugger-mono

Milestone: -

@thaystg thaystg changed the title [release/7.0] Indexing with expression [release/7.0] [wasm][debugger] Indexing with expression Sep 13, 2022
@thaystg thaystg changed the title [release/7.0] [wasm][debugger] Indexing with expression [release/7.0][wasm][debugger] Indexing with expression Sep 13, 2022
@carlossanlop
Copy link
Contributor

The main PR needs to get merged first. I'll add the "No merge" label. Please remove it when that PR is merged.

@carlossanlop carlossanlop added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Sep 13, 2022
@radical
Copy link
Member

radical commented Sep 14, 2022

@carlossanlop we can merge this before the main PR is done. I added some test changes to the main one, but nothing material that would be needed here.

@carlossanlop carlossanlop removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Sep 14, 2022
@carlossanlop
Copy link
Contributor

Ok no problem, @radical. Let's get this in. :shipit:

@carlossanlop carlossanlop merged commit ae02fc8 into release/7.0 Sep 14, 2022
@carlossanlop carlossanlop deleted the backport/pr-75524-to-release/7.0 branch September 14, 2022 00:43
@ghost ghost locked as resolved and limited conversation to collaborators Oct 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants