Skip to content

Conversation

isra-fel
Copy link
Member

Added exception for plural parameter names regarding units.

Description

Clarify guidelines on plural parameter names

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

Added exception for plural parameter names regarding units.
@Copilot Copilot AI review requested due to automatic review settings September 23, 2025 06:53
Copy link

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR clarifies the Azure PowerShell design guidelines regarding plural parameter names by adding an exception for units. The change addresses a common scenario where parameter names naturally use plural forms when indicating units of measurement.

  • Adds clarification that units are an exception to the plural parameter naming guideline

@VeryEarly VeryEarly merged commit 451e763 into main Sep 23, 2025
12 checks passed
@VeryEarly VeryEarly deleted the isra-fel-patch-1 branch September 23, 2025 08:29
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