| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
Ilya (@iSazonov) |
Sorry, something went wrong.
|
Paul Higinbotham (@PaulHigin) I updated a link to the issue. |
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM
Sorry, something went wrong.
|
🤔 makes me wonder if we're not better off moving the emphasis method out of the MatchInfo class itself and make it a helper method on a utils class somewhere instead. |
Sorry, something went wrong.
|
Rain Sallow (/u/ta11ow) (@vexx32) The fix is simple and we could include it in 7.0.1. Moving the emphasis method is more complex and we can break something. We need to think about this more. |
Sorry, something went wrong.
|
🎉v7.1.0-preview.1 has been released which incorporates this pull request.:tada: Handy links: |
Sorry, something went wrong.
MatchInfo class has ToEmphasizedString() member method to color output line. In remote scenario MatchInfo class is deserialized as Deserialized.Microsoft.PowerShell.Commands.MatchInfo without the method. As result default formating in remote scenario show nothing. The fix is to directly output Line property.
| Back | FazBrowse Home | New Git URL |
PR Summary
Fix #3480
MatchInfo class has ToEmphasizedString() member method to color output line.
In remote scenario MatchInfo class is deserialized as Deserialized.Microsoft.PowerShell.Commands.MatchInfo without the method. As result default formating in remote scenario show nothing.
The fix is to directly output Line property.
No tests are added because we haven't remote CI tests. Tested manually.
PR Context
PR Checklist