Skip to content
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

Make the use of Oxford commas even more consistent #25140

Merged
merged 1 commit into from
Mar 10, 2025

Conversation

pressRtowin
Copy link
Contributor

@pressRtowin pressRtowin commented Mar 7, 2025

ManagedEntranceStrings.resx has a comma here for the exact same string. Just making the two consistent.

PR Summary

Added two commas

PR Context

The commas are consistent with the previous use of a comma in ManagedEntranceStrings.resx and my proposed additional comma there in #25139.

PR Checklist

ManagedEntranceStrings.resx has a comma here for the exact same line. Just making the two consistent.
@iSazonov iSazonov requested a review from sdwheeler March 8, 2025 06:53
@iSazonov iSazonov added the CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log label Mar 8, 2025
Copy link
Collaborator

@sdwheeler sdwheeler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@iSazonov
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@iSazonov iSazonov enabled auto-merge (squash) March 10, 2025 16:21
@iSazonov iSazonov merged commit dc6b4c2 into PowerShell:master Mar 10, 2025
38 of 40 checks passed
Copy link
Contributor

microsoft-github-policy-service bot commented Mar 10, 2025

📣 Hey @pressRtowin, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗 https://aka.ms/PSRepoFeedback

@iSazonov iSazonov self-assigned this Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants