-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Snap 17.7p3 #68821
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
RikkiGibson
merged 823 commits into
release/dev17.7
from
merges/main-to-release/dev17.7
Jun 28, 2023
Merged
Snap 17.7p3 #68821
RikkiGibson
merged 823 commits into
release/dev17.7
from
merges/main-to-release/dev17.7
Jun 28, 2023
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Move off of recursion when determining pp-directive information for a file.
Switch to new lower-overhead spell checking protocol
…-cleanups-and-perf-improvements Ensure we only ask for a symbol's documentation comment once
…ywordName Fix 'use primary constructor' with nested types.
Fix 'Use Local Function' fixer in code style layer
Use DelayTimeSpan where applicable
Fixes random test failures involving cancellation of ServiceHub requests.
Update to Nerdbank.Streams 2.9.116
…host Merge internal fork of LSP tools back into Roslyn
…ences (#66515) * fixed CA1854 occurrences c# * fixed CA1854 occurrences VB * VB init with Nothing + fix formatting * Address PR feedback * Update src/Workspaces/Core/Portable/Rename/ConflictEngine/RenamedSpansTracker.cs Co-authored-by: Jan Jones <jan.jones.cz@gmail.com> * remove redundant ifs * Apply suggestions from code review Co-authored-by: CyrusNajmabadi <cyrus.najmabadi@gmail.com> --------- Co-authored-by: Jan Jones <jan.jones.cz@gmail.com> Co-authored-by: CyrusNajmabadi <cyrus.najmabadi@gmail.com>
Roslyn side changes to enable full solution background analysis in LSP/VSCode
…626.5 (#68802) [main] Update dependencies from dotnet/arcade - Coherency Updates: - Microsoft.DotNet.XliffTasks: from 1.0.0-beta.23312.2 to 1.0.0-beta.23319.1 (parent: Microsoft.DotNet.Arcade.Sdk)
Should address our side of https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1837883
…yPackageVersion Update VSTelemetry package version
"Fix" LSP semantic token modifiers
Avoid using 'solution crawler' terminology in the option name
Rename background analysis option name
arunchndr
approved these changes
Jun 28, 2023
* Fix SuppressMessage attributes working on primary constructors * Simplify * Simplify * Use helper * update docs * update docs * Add dedicated tests * SHare code, add test * remove using * Rename
Closes #68819. Consumption of such inline array types is still not supported.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.