Skip to content

Merge for 17.3 Preview 2 snap#61674

Merged
JoeRobich merged 1105 commits intorelease/dev17.3from
merge-for-snap
Jun 3, 2022
Merged

Merge for 17.3 Preview 2 snap#61674
JoeRobich merged 1105 commits intorelease/dev17.3from
merge-for-snap

Conversation

@JoeRobich
Copy link
Copy Markdown
Member

The original changes were squash merged into the release branch.

jcouv and others added 30 commits May 13, 2022 10:55
It's valid for an input to be modified causing a downstream input to be removed (for example a syntax tree can change what is in it, leading to the downstream node not generating something). 

//cc @jkoritzinsky in case I'm missing something obvious.
More SyntaxContext sharing among CompletionProviders
* Fix several LSP completion kind mappings

* Extension method
Fix crash when a multi-line raw literal becomes a skipped token.
Add an UWP OptProf test for IDE
…rmatter. (#60522)

* Implement IDkmClrFullNameProvider2 in Roslyn's ResultProvider Formatter.

Issue:
- Debugger added IDkmClrFullNameProvider2 API with https://devdiv.visualstudio.com/DevDiv/_git/Concord/pullrequest/301518
- It is currently temporarily implement in Concord, needs to be implemented in Roslyn so implementation can be removed from Concord.

Changes:
1. Formatter:
- Implement IDkmClrFullNameProvider2. It has 2 methods, one to format local names and the other given field metadata.
- Currently only implemented for C#. I'm not that familiar with VB and the GeneratedNames stuff in VB needs some splitting and moving around to get working.

2. Unit tests:
- Add unit tests for the common cases of hoisted locals, synthesized locals, etc.

3. Versions.props:
Update MicrosoftVSSDKVSDConfigToolVersion to a newer version which recognizes IDkmClrFullNameProvider2.

* PR feedback - fix casing of MetadataImport

* PR feedback - move GetOriginalLocalVariableName, GetOriginalFieldName back to CSharpFormatter

Co-authored-by: Ramkumar Ramesh <ramram@microsoft.com>
Show not keyword (and all other stuff) after and & or when they are treated as identifiers
…d-members

Merge main to features/required-members
333fred and others added 26 commits May 27, 2022 16:52
…quence

Add some APIs on AspNetCoreVirtualCharSequence
…ss-ivt

Add unit test project IVT to ExternalAccess.AspNetCore
InlineHintsTagger: Fix assumption that GetSpans will retrieve a length greater than 0
Fix generation location when generating across files
…1356)

* Implements support for adding and removing await/yield return in C# async, iterator and async iterator methods.

* Fix syntax node associated with BoundTryStatement created from using syntax
@ghost ghost added the Area-Infrastructure label Jun 3, 2022
@JoeRobich JoeRobich merged commit 43bacfe into release/dev17.3 Jun 3, 2022
@JoeRobich JoeRobich deleted the merge-for-snap branch May 7, 2024 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.