Skip to content

Types cleanup - #2076

Merged
adamsitnik merged 6 commits into
dotnet:mainfrom
adamsitnik:cleanupp
Mar 7, 2023
Merged

Types cleanup#2076
adamsitnik merged 6 commits into
dotnet:mainfrom
adamsitnik:cleanupp

Conversation

@adamsitnik

Copy link
Copy Markdown
Member

No description provided.

@adamsitnik
adamsitnik requested review from Keboo and jonsequitur March 6, 2023 14:22
/// </summary>
public class CompletionItem : IEquatable<CompletionItem>
{
// reference: https://microsoft.github.io/language-server-protocol/specifications/specification-3-17/#textDocument_completion

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Comment thread src/System.CommandLine/Completions/CompletionItem.cs Outdated
internal static void SetTerminalForegroundRed(this IConsole console)
{
if (Platform.IsConsoleRedirectionCheckSupported &&
if (IsConsoleRedirectionCheckSupported &&

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Related bug #1851.

@adamsitnik
adamsitnik merged commit bf523f8 into dotnet:main Mar 7, 2023
@adamsitnik
adamsitnik deleted the cleanupp branch March 7, 2023 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants