Skip to content

Fix quickinfo text - #17682

Merged
KevinRansom merged 5 commits into
dotnet:mainfrom
KevinRansom:quickinfomain
Sep 11, 2024
Merged

Fix quickinfo text#17682
KevinRansom merged 5 commits into
dotnet:mainfrom
KevinRansom:quickinfomain

Conversation

@KevinRansom

Copy link
Copy Markdown
Contributor

No description provided.

@KevinRansom
KevinRansom requested a review from a team as a code ownerSeptember 10, 2024 02:19
@github-actions

github-actionsBot commented Sep 10, 2024

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

@KevinRansom

KevinRansom commented Sep 10, 2024

Copy link
Copy Markdown
ContributorAuthor

Passed
Failed
Failed
Failed
Failed
Failed

@KevinRansomKevinRansom added the NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes label Sep 10, 2024
@KevinRansomKevinRansom self-assigned this Sep 10, 2024
@KevinRansomKevinRansom added this to the September-2024 milestone Sep 10, 2024
@majocha

Copy link
Copy Markdown
Contributor

Probably it would be worth it to also test this case in FSharp.Compiler.Service.Tests.TooltipTests.

@KevinRansom

Copy link
Copy Markdown
ContributorAuthor

Probably it would be worth it to also test this case in FSharp.Compiler.Service.Tests.TooltipTests.

I just want the durned thing to work reliably again. :-)

@psfinaki

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@KevinRansom
KevinRansomforce-pushed the quickinfomain branch 3 times, most recently from 16cd082 to efc01b2CompareSeptember 11, 2024 07:05
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

5 participants

@KevinRansom@majocha@psfinaki@vzarytovskii@T-Gro