Skip to content

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

Merged
carlossanlop merged 3 commits into
release/7.0from
backport/pr-75524-to-release/7.0
Sep 14, 2022
Merged

[release/7.0][wasm][debugger] Indexing with expression #75559
carlossanlop merged 3 commits into
release/7.0from
backport/pr-75524-to-release/7.0

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Sep 13, 2022

Copy link
Copy Markdown
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

ghost commented Sep 13, 2022

Copy link
Copy Markdown

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:-

@thaystgthaystg changed the title [release/7.0] Indexing with expression [release/7.0] [wasm][debugger] Indexing with expression Sep 13, 2022
@thaystgthaystg changed the title [release/7.0] [wasm][debugger] Indexing with expression [release/7.0][wasm][debugger] Indexing with expression Sep 13, 2022
@carlossanlop

carlossanlop commented Sep 13, 2022

Copy link
Copy Markdown
Contributor

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

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

radical commented Sep 14, 2022

Copy link
Copy Markdown
Member

@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.

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

carlossanlop commented Sep 14, 2022

Copy link
Copy Markdown
Contributor

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

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

Reviewers

@radicalradicalradical approved these changes
@lewinglewinglewing approved these changes
@steveisoksteveisoksteveisok approved these changes
@thaystgthaystgAwaiting requested review from thaystgthaystg is a code owner
@marek-safarmarek-safarAwaiting requested review from marek-safar

Assignees

No one assigned

Labels

area-Debugger-mono

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@carlossanlop@radical@lewing@steveisok@ilonatommy