Skip to content

Conversation

Taojunshen
Copy link
Contributor

@Taojunshen Taojunshen commented Sep 9, 2024

Prior to this change, the about_Quoting_Rules topic document didn't indicate that PowerShell treats smart/typographic quotation marks as normal/programmer quotation marks, making them subject to the same escaping and quoting behaviors.

This change:

  • Adds a note about smart quotation marks and guidance to avoid using them to enclose strings.
  • Adds an example of escaping smart quotation marks.
  • Fixes AB#309820
  • Resolves Note about smart quotes #11401

PR Summary

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributors guide.
  • Style: This PR adheres to the style guide.

Prior to this change, the `about_Quoting_Rules` topic document
didn't indicate that PowerShell treats smart/typographic
quotation marks as normal/programmer quotation marks, making
them subject to the same escaping and quoting behaviors.

This change:

- Adds a note about smart quotation marks and guidance to avoid
  using them to enclose strings.
- Adds an example of escaping smart quotation marks.
- Fixes AB#309820
- Resolves #11401
Copy link
Contributor

Learn Build status updates of commit 46df435:

✅ Validation status: passed

File Status Preview URL Details
reference/5.1/Microsoft.PowerShell.Core/About/about_Quoting_Rules.md ✅Succeeded View (powershell-5.1)
reference/7.2/Microsoft.PowerShell.Core/About/about_Quoting_Rules.md ✅Succeeded View (powershell-7.2)
reference/7.4/Microsoft.PowerShell.Core/About/about_Quoting_Rules.md ✅Succeeded View (powershell-7.4)
reference/7.5/Microsoft.PowerShell.Core/About/about_Quoting_Rules.md ✅Succeeded View (powershell-7.5)

For more details, please refer to the build report.

For any questions, please:

@Taojunshen Taojunshen merged commit a5ae33c into live Sep 9, 2024
16 checks passed
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.

2 participants