Pester icon indicating copy to clipboard operation
Pester copied to clipboard

Don't throw legacy error when using Should operator after escaped linebreak

Open fflaten opened this issue 2 years ago • 2 comments

PR Summary

Fix false positive exception when specifying Should-operator after linebreak using backtick/escape char, ex.

1 | Should `
    -Be 1

Provided as an alternative to removing the legacy-check though I'm not against removing it.

PR Checklist

  • [x] PR has meaningful title
  • [x] Summary describes changes
  • [ ] PR is ready to be merged
    • If not, use the arrow next to Create Pull Request to mark it as a draft. PR can be marked Ready for review when it's ready.
  • [x] Tests are added/update (if required)
  • [ ] Documentation is updated/added (if required)

fflaten avatar Mar 23 '23 21:03 fflaten

/azp run

fflaten avatar Mar 25 '23 12:03 fflaten

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 25 '23 12:03 azure-pipelines[bot]

Added note to remove the whole syntax in 6.0.0 here https://github.com/pester/Pester/issues/2437

Do you still want to fix this, or close?

nohwnd avatar Apr 12 '24 19:04 nohwnd