Powershell: fix string escape issue #3661
Conversation
|
Thank you for submitting the pull request. Please, read https://docs.ctags.io/en/latest/contributions.html#squashing-commits . I don't want to merge your try-and-error commit sequence. Please, read https://docs.ctags.io/en/latest/contributions.html#title-of-commit-log-and-pull-request Maybe "Powershell: " is the suitable prefix.
Thank you. This is important information for the current reviewers and future reviewers. Thank you. @kumarstack55 Can I ask you to look into this pull request? |
|
OK! Let me update the PR. |
iaalm
changed the title
Powershell use ` instead of \ to escape string. Current implement ignores all code after something like "\". ref: https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_special_characters?view=powershell-7.3
|
Thank you. |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #3661 +/- ##
=======================================
Coverage 82.83% 82.83%
=======================================
Files 223 223
Lines 54508 54508
=======================================
Hits 45149 45149
Misses 9359 9359
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
|
@kumarstack55 could you help reviewing this PR? |
According to official doc, powershell use ` to escape string and it only works in ".