Artifact Signing module docs cleanup by Jaxelr · Pull Request #29262 · Azure/azure-powershell

Jaxelr · 2026-03-11T02:40:15Z

Description

This is a follow up to the PR #29104 where it does a bit of cleanup of inconsistent code comments or error messages.

Mandatory Checklist

Please choose the target release of Azure PowerShell. (⚠️Target release is a different concept from API readiness. Please click below links for details.)
- General release
- Public preview
- Private preview
- Engineering build
- No need for a release
Check this box to confirm: I have read the Submitting Changes section of CONTRIBUTING.md and reviewed the following information:

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

azure-client-tools-bot-prd · 2026-03-11T02:40:18Z

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

Copilot

Pull request overview

This PR is a follow-up cleanup for the Az.ArtifactSigning module, focusing on making documentation and user-facing strings consistently refer to “artifact signing” (instead of “trusted signing”), plus a few small generated-file updates.

Changes:

Updated multiple help markdown files (both help/ and ArtifactSigning.Autorest/docs/) to replace “trusted signing” wording with “artifact signing”.
Refreshed module packaging metadata (changelog entry, manifest formatting, AutoRest pinned commit, generate-info ID).
Minor code cleanup (removed a commented-out debug line; adjusted a cmdlet HelpMessage string).

Reviewed changes

Copilot reviewed 26 out of 26 changed files in this pull request and generated 9 comments.

Show a summary per file

File	Description
src/ArtifactSigning/ArtifactSigning/help/Update-AzArtifactSigningAccount.md	Wording updates to “artifact signing” in cmdlet help.
src/ArtifactSigning/ArtifactSigning/help/Test-AzArtifactSigningAccountNameAvailability.md	Wording/casing updates in cmdlet help examples and descriptions.
src/ArtifactSigning/ArtifactSigning/help/Remove-AzArtifactSigningCertificateProfile.md	Parameter description wording update.
src/ArtifactSigning/ArtifactSigning/help/Remove-AzArtifactSigningAccount.md	Wording updates to “artifact signing” in help content.
src/ArtifactSigning/ArtifactSigning/help/New-AzArtifactSigningCertificateProfile.md	Parameter description wording update.
src/ArtifactSigning/ArtifactSigning/help/New-AzArtifactSigningAccount.md	Wording updates to “artifact signing” in help content.
src/ArtifactSigning/ArtifactSigning/help/Get-AzArtifactSigningCertificateProfile.md	Parameter description wording update.
src/ArtifactSigning/ArtifactSigning/help/Get-AzArtifactSigningAccount.md	Wording updates to “artifact signing” in help content.
src/ArtifactSigning/ArtifactSigning/help/Az.ArtifactSigning.md	Module index synopsis lines updated to “artifact signing”.
src/ArtifactSigning/ArtifactSigning/Helpers/CmsSigner.cs	Removed commented-out debug output line.
src/ArtifactSigning/ArtifactSigning/Commands/InvokeCIPolicySigning.cs	Updated user-facing HelpMessage text to “ArtifactSigning”.
src/ArtifactSigning/ArtifactSigning/ChangeLog.md	Added upcoming-release note describing doc/comment cleanup.
src/ArtifactSigning/ArtifactSigning/Az.ArtifactSigning.psd1	Manifest regenerated/normalized (timestamps, formatting, empty arrays).
src/ArtifactSigning/ArtifactSigning.Autorest/generate-info.json	Updated generation ID.
src/ArtifactSigning/ArtifactSigning.Autorest/docs/Update-AzArtifactSigningAccount.md	Wording updates to “Artifact Signing” in generated docs.
src/ArtifactSigning/ArtifactSigning.Autorest/docs/Test-AzArtifactSigningAccountNameAvailability.md	Wording/casing updates in generated docs.
src/ArtifactSigning/ArtifactSigning.Autorest/docs/Remove-AzArtifactSigningCertificateProfile.md	Wording and example text updates in generated docs.
src/ArtifactSigning/ArtifactSigning.Autorest/docs/Remove-AzArtifactSigningAccount.md	Wording updates in generated docs.
src/ArtifactSigning/ArtifactSigning.Autorest/docs/New-AzArtifactSigningCertificateProfile.md	Wording updates in generated docs.
src/ArtifactSigning/ArtifactSigning.Autorest/docs/New-AzArtifactSigningAccount.md	Wording updates in generated docs.
src/ArtifactSigning/ArtifactSigning.Autorest/docs/Get-AzArtifactSigningCertificateProfile.md	Wording updates in generated docs.
src/ArtifactSigning/ArtifactSigning.Autorest/docs/Get-AzArtifactSigningAccount.md	Wording updates in generated docs.
src/ArtifactSigning/ArtifactSigning.Autorest/docs/Az.ArtifactSigning.md	Module-level generated docs updated (including Module Guid line).
src/ArtifactSigning/ArtifactSigning.Autorest/UX/Microsoft.CodeSigning/codeSigningAccounts.json	Updated command descriptions/example descriptions to “Artifact Signing”.
src/ArtifactSigning/ArtifactSigning.Autorest/README.md	Updated pinned swagger/spec commit hash.
src/ArtifactSigning/ArtifactSigning.Autorest/Properties/AssemblyInfo.cs	Assembly version/file version updated.

Copilot · 2026-03-11T02:45:52Z

src/ArtifactSigning/ArtifactSigning/help/Az.ArtifactSigning.md


 ### [Get-AzArtifactSigningCustomerEku](Get-AzArtifactSigningCustomerEku.md)
-Retrieve Azure.CodeSigning customer Eku
+Retrieve artifact signing customer Eku


Eku should be capitalized as the acronym EKU (Enhanced Key Usage) in this synopsis line to avoid looking like a misspelling.

Suggested change

Retrieve artifact signing customer Eku

Retrieve artifact signing customer EKU

Copilot · 2026-03-11T02:45:53Z

src/ArtifactSigning/ArtifactSigning/help/Az.ArtifactSigning.md


 ### [New-AzArtifactSigningAccount](New-AzArtifactSigningAccount.md)
-create a trusted Signing Account.
+create an artifact signing account.


This synopsis line starts with a lowercase verb (create). For consistency with the other entries in this module index (and standard sentence casing), capitalize it (e.g., Create ...).

Copilot · 2026-03-11T02:45:53Z

src/ArtifactSigning/ArtifactSigning/help/Test-AzArtifactSigningAccountNameAvailability.md

 ## EXAMPLES

-### Example 1: Test The Availability Of An Used Artifact signing Account Name
+### Example 1: Test The Availability Of An Used Artifact Signing Account Name


The example heading uses incorrect grammar: An Used should be reworded (e.g., a used / an already-used / an in-use) to be grammatically correct.

Suggested change

### Example 1: Test The Availability Of An Used Artifact Signing Account Name

### Example 1: Test The Availability Of A Used Artifact Signing Account Name

Copilot · 2026-03-11T02:45:53Z

src/ArtifactSigning/ArtifactSigning/help/Test-AzArtifactSigningAccountNameAvailability.md

-### Example 1: Test The Availability Of An Used Artifact signing Account Name
+### Example 1: Test The Availability Of An Used Artifact Signing Account Name
 ```powershell
 Test-AzArtifactSigningAccountNameAvailability -Name unavaliable


The example uses unavaliable as the name value; this is a misspelling of unavailable and should be corrected to avoid propagating the typo into user scripts.

Copilot · 2026-03-11T02:45:53Z

...tifactSigning/ArtifactSigning.Autorest/docs/Test-AzArtifactSigningAccountNameAvailability.md

 ## EXAMPLES

-### Example 1: Test The Availability Of An Used Artifact signing Account Name
+### Example 1: Test The Availability Of An Used Artifact Signing Account Name


The example heading uses incorrect grammar: An Used should be reworded (e.g., a used / an already-used / an in-use) to be grammatically correct.

Suggested change

### Example 1: Test The Availability Of An Used Artifact Signing Account Name

### Example 1: Test The Availability Of a Used Artifact Signing Account Name

Copilot · 2026-03-11T02:45:54Z

...tifactSigning/ArtifactSigning.Autorest/docs/Test-AzArtifactSigningAccountNameAvailability.md

-### Example 1: Test The Availability Of An Used Artifact signing Account Name
+### Example 1: Test The Availability Of An Used Artifact Signing Account Name
 ```powershell
 Test-AzArtifactSigningAccountNameAvailability -Name unavaliable


The example uses unavaliable as the name value; this is a misspelling of unavailable and should be corrected to avoid propagating the typo into user scripts.

Copilot · 2026-03-11T02:45:54Z

src/ArtifactSigning/ArtifactSigning.Autorest/docs/Az.ArtifactSigning.md

 ---
 Module Name: Az.ArtifactSigning
-Module Guid: ff7c6523-4ad4-433c-9cb6-568e4b2daff2
+Module Guid: 323532e3-ac50-4624-8d7e-91b9a220d6a2
 Download Help Link: https://learn.microsoft.com/powershell/module/az.artifactsigning


The Module Guid value in this doc appears to have changed. Module GUIDs are typically treated as stable identifiers for a module’s documentation; please confirm this is intentional and, if not, revert it (or align it with the module’s actual manifest GUID).

Copilot · 2026-03-11T02:45:54Z

src/ArtifactSigning/ArtifactSigning/help/Az.ArtifactSigning.md


 ### [Update-AzArtifactSigningAccount](Update-AzArtifactSigningAccount.md)
-update a trusted signing account.
+update an artifact signing account.


This synopsis line starts with a lowercase verb (update). For consistency with the other entries in this module index (and standard sentence casing), capitalize it (e.g., Update ...).

Copilot · 2026-03-11T02:45:54Z

src/ArtifactSigning/ArtifactSigning/Commands/InvokeCIPolicySigning.cs

            ParameterSetName = ByAccountProfileNameParameterSet,
            ValueFromPipelineByPropertyName = true,
-            HelpMessage = "The endpoint url used to submit request to Azure TrustedSigning.")]
+            HelpMessage = "The endpoint url used to submit request to Azure ArtifactSigning.")]


This HelpMessage still uses url instead of URL. Since this string is shown to users, please use standard capitalization (and consider applying the same change to the other related HelpMessage strings in this cmdlet for consistency).

Suggested change

HelpMessage = "The endpoint url used to submit request to Azure ArtifactSigning.")]

HelpMessage = "The endpoint URL used to submit request to Azure ArtifactSigning.")]

NoriZC · 2026-03-11T03:08:41Z

/azp run

azure-pipelines · 2026-03-11T03:08:56Z

Azure Pipelines successfully started running 3 pipeline(s).

NoriZC · 2026-03-14T01:50:42Z

/azp run

azure-pipelines · 2026-03-14T01:50:57Z

Azure Pipelines successfully started running 3 pipeline(s).

Jaxelr added 3 commits March 10, 2026 19:26

fix: update artifact signing comments and messages

2de2ffa

fix: revert sln changes

a8b31cc

fix: revert auto generated code

ce458f9

Copilot AI review requested due to automatic review settings March 11, 2026 02:40

docs: update changelog.md

c8d70ca

Copilot started reviewing on behalf of Jaxelr March 11, 2026 02:42 View session

Copilot AI reviewed Mar 11, 2026

View reviewed changes

Update ChangeLog.md

9d00f43

VeryEarly approved these changes Mar 14, 2026

View reviewed changes

VeryEarly enabled auto-merge (squash) March 14, 2026 02:07

VeryEarly merged commit 2995ff0 into Azure:main Mar 14, 2026
12 checks passed

	Retrieve artifact signing customer Eku
	Retrieve artifact signing customer EKU

	### Example 1: Test The Availability Of An Used Artifact Signing Account Name
	### Example 1: Test The Availability Of A Used Artifact Signing Account Name

	HelpMessage = "The endpoint url used to submit request to Azure ArtifactSigning.")]
	HelpMessage = "The endpoint URL used to submit request to Azure ArtifactSigning.")]

Conversation

Jaxelr commented Mar 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Mandatory Checklist

Uh oh!

azure-client-tools-bot-prd bot commented Mar 11, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

NoriZC commented Mar 11, 2026

Uh oh!

azure-pipelines bot commented Mar 11, 2026

Uh oh!

NoriZC commented Mar 14, 2026

Uh oh!

azure-pipelines bot commented Mar 14, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Jaxelr commented Mar 11, 2026 •

edited

Loading