-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
Support mapping foreground/background ConsoleColor
values to VT escape sequences
#17938
Conversation
VTUtility pre-dates |
That sounds totally reasonable to me. I can move the changes over to |
@SteveL-MSFT The changes are moved to |
@daxian-dbw it seems logical and may improve discovery to have |
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
Agreed. Added the |
VTUtility
to support mapping foreground/background ConsoleColor
values to VT escape sequencesConsoleColor
values to VT escape sequences
src/Microsoft.PowerShell.Commands.Utility/commands/utility/MatchString.cs
Show resolved
Hide resolved
🎉 Handy links: |
PR Summary
Fix: #17937
Update
PSStyle
to support mapping foreground/backgroundConsoleColor
values to VT escape sequences, and mark theVTUtility
class deprecated.PR Context
Please see the detailed description in #17937 for the context.
PR Checklist
.h
,.cpp
,.cs
,.ps1
and.psm1
files have the correct copyright headerWIP:
or[ WIP ]
to the beginning of the title (theWIP
bot will keep its status check atPending
while the prefix is present) and remove the prefix when the PR is ready.