Skip to content

chore(Async.RunSynchronouslyImmediate): Stragglers re #19804 - #20245

Merged
T-Gro merged 3 commits into
dotnet:mainfrom
bartelink:rsi-stragglers
Aug 12, 2026
Merged

chore(Async.RunSynchronouslyImmediate): Stragglers re #19804#20245
T-Gro merged 3 commits into
dotnet:mainfrom
bartelink:rsi-stragglers

Conversation

@bartelink

@bartelinkbartelink commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Followups to fsharp/fslang-suggestions#1042 (from fsharp/fslang-suggestions#1042) via #19844 (comment)

  • rename and sync clone impls within VisualFSharp.slnx as per previous PR
  • update internal helper RunImmediateExceptOnUI to delegate to RSI and follow naming

Checklist

  • Release notes entry updated (PR ref added to existing changelog entry for FSharp.CompilerService

@github-actions

github-actionsBot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

⚠️ Release notes required, but author opted out

Warning

Author opted out of release notes, check is disabled for this pull request.
cc @dotnet/fsharp-team-msft

@bartelinkbartelink changed the title chore(Async.RunSynchronouslyImmediate): Stragglers from #19804chore(Async.RunSynchronouslyImmediate): Stragglers re #19804Aug 11, 2026
@bartelink
bartelink marked this pull request as ready for review August 11, 2026 15:51
@bartelink
bartelink requested a review from a team as a code ownerAugust 11, 2026 15:51
CopilotAI lite review requested due to automatic review settings August 11, 2026 15:51
@bartelink

Copy link
Copy Markdown
ContributorAuthor

@T-Gro want me to propagate that changelog entry to VS release notes like the comment says?

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR finishes the Async.RunImmediateAsync.RunSynchronouslyImmediate rename across VS integration code/tests and updates helper APIs to match the new naming and behavior (notably exception unwrapping).

Changes:

  • Replace remaining Async.RunImmediate usages in VS integration and Salsa tests with Async.RunSynchronouslyImmediate.
  • Rename RunImmediateExceptOnUI to RunSynchronouslyImmediateExceptOnUI and delegate to Async.RunSynchronouslyImmediate.
  • Update benchmarks/source payloads and release notes to reflect the rename.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file
FileDescription
vsintegration/tests/Salsa/salsa.fsRename call sites to Async.RunSynchronouslyImmediate.
vsintegration/tests/Salsa/FSharpLanguageServiceTestable.fsRename NotifyProjectCleaned call site to Async.RunSynchronouslyImmediate.
vsintegration/tests/FSharp.Editor.Tests/BraceMatchingServiceTests.fsUpdate tests to use RunSynchronouslyImmediateExceptOnUI.
vsintegration/src/FSharp.LanguageService/LanguageServiceConstants.fsIntroduces a shim for RunSynchronouslyImmediate (currently problematic).
vsintegration/src/FSharp.LanguageService/FSharpSource.fsRename parse call site to Async.RunSynchronouslyImmediate.
vsintegration/src/FSharp.LanguageService/BackgroundRequests.fsRename various checker call sites to Async.RunSynchronouslyImmediate.
vsintegration/src/FSharp.Editor/Common/Extensions.fsRename helper and delegate to Async.RunSynchronouslyImmediate off-UI thread.
tests/benchmarks/FCSBenchmarks/CompilerServiceBenchmarks/SomethingToCompileSmaller.fsUpdates embedded source; currently introduces a syntax error + duplicate member.
tests/benchmarks/FCSBenchmarks/CompilerServiceBenchmarks/SomethingToCompile.fsUpdates embedded source; currently defines a duplicate member.
docs/release-notes/.FSharp.Compiler.Service/11.0.100.mdAdds PR reference to existing entry (minor formatting issue).

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Comment threadvsintegration/src/FSharp.LanguageService/LanguageServiceConstants.fs Outdated
Comment threaddocs/release-notes/.FSharp.Compiler.Service/11.0.100.md Outdated
@github-actionsgithub-actionsBot added the AI-Tooling-Check-Scanned-Clean Tooling check: diff analyzed, no interesting infrastructure files label Aug 11, 2026
- rename and sync clone impls withing VisualFSharp.slnx as per previous PR
- update RunImmediateExceptOnUI to delegate and follow naming
@T-GroT-Gro added the NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes label Aug 12, 2026
@T-Gro
T-Gro merged commit 1852df8 into dotnet:mainAug 12, 2026
48 of 49 checks passed
@bartelink
bartelink deleted the rsi-stragglers branch August 12, 2026 09:12
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI-Tooling-Check-Scanned-CleanTooling check: diff analyzed, no interesting infrastructure filesNO_RELEASE_NOTESLabel for pull requests which signals, that user opted-out of providing release notes

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants

@bartelink@T-Gro