Skip to content

[NetAppFiles] Bump to 2025-12-01#29182

Open
audunn wants to merge 9 commits intoAzure:mainfrom
audunn:relase-Microsoft.NetApp-2025-12-01
Open

[NetAppFiles] Bump to 2025-12-01#29182
audunn wants to merge 9 commits intoAzure:mainfrom
audunn:relase-Microsoft.NetApp-2025-12-01

Conversation

@audunn
Copy link
Member

@audunn audunn commented Feb 18, 2026

Description

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

Copilot AI review requested due to automatic review settings February 18, 2026 14:52
@azure-client-tools-bot-prd
Copy link

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

Copy link
Contributor

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.

Copilot reviewed 116 out of 190 changed files in this pull request and generated 4 comments.

Files not reviewed (1)
  • src/NetAppFiles/NetAppFiles/Properties/Resources.Designer.cs: Language not supported

@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

Copilot AI review requested due to automatic review settings February 23, 2026 10:27
@audunn
Copy link
Member Author

audunn commented Feb 23, 2026

/azp run

@azure-pipelines
Copy link
Contributor

Commenter does not have sufficient privileges for PR 29182 in repo Azure/azure-powershell

Copy link
Contributor

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

Copilot reviewed 116 out of 190 changed files in this pull request and generated 3 comments.

Files not reviewed (1)
  • src/NetAppFiles/NetAppFiles/Properties/Resources.Designer.cs: Language not supported

@audunn
Copy link
Member Author

audunn commented Feb 23, 2026

/azp run

@azure-pipelines
Copy link
Contributor

Commenter does not have sufficient privileges for PR 29182 in repo Azure/azure-powershell

Copilot AI review requested due to automatic review settings February 23, 2026 13:05
@audunn
Copy link
Member Author

audunn commented Feb 23, 2026

/azp run

@azure-pipelines
Copy link
Contributor

Commenter does not have sufficient privileges for PR 29182 in repo Azure/azure-powershell

Copy link
Contributor

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

Copilot reviewed 116 out of 190 changed files in this pull request and generated 3 comments.

Files not reviewed (1)
  • src/NetAppFiles/NetAppFiles/Properties/Resources.Designer.cs: Language not supported

Comment on lines +356 to +358
### -ProgressAction
{{ Fill ProgressAction Description }}

Copy link

Copilot AI Feb 23, 2026

Choose a reason for hiding this comment

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

The new -ProgressAction parameter section still contains the placeholder text {{ Fill ProgressAction Description }}. This will surface in published help and should be replaced with a real description (for example, the standard ActionPreference wording used in other NetAppFiles help topics) before merging. This placeholder appears to have been introduced across multiple updated help files in this PR, so it should be updated consistently everywhere it was added.

Copilot uses AI. Check for mistakes.
Comment on lines +80 to +82
### -ProgressAction
{{ Fill ProgressAction Description }}

Copy link

Copilot AI Feb 23, 2026

Choose a reason for hiding this comment

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

The -ProgressAction parameter description is still the placeholder {{ Fill ProgressAction Description }}. Please replace it with a real description (for example, the standard ActionPreference wording used in other help topics) before merging.

Copilot uses AI. Check for mistakes.
Comment on lines +177 to +179
### -ProgressAction
{{ Fill ProgressAction Description }}

Copy link

Copilot AI Feb 23, 2026

Choose a reason for hiding this comment

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

The newly added -ProgressAction parameter section uses the placeholder {{ Fill ProgressAction Description }}. This should be replaced with a real description so the generated/online help doesn't ship with template text.

Copilot uses AI. Check for mistakes.
@audunn
Copy link
Member Author

audunn commented Feb 23, 2026

/azp run

@azure-pipelines
Copy link
Contributor

Commenter does not have sufficient privileges for PR 29182 in repo Azure/azure-powershell

@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@audunn
Copy link
Member Author

audunn commented Feb 25, 2026

/azp run

@azure-pipelines
Copy link
Contributor

Commenter does not have sufficient privileges for PR 29182 in repo Azure/azure-powershell

Copilot AI review requested due to automatic review settings February 25, 2026 16:41
Copy link
Contributor

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

Copilot reviewed 117 out of 192 changed files in this pull request and generated 4 comments.

Files not reviewed (1)
  • src/NetAppFiles/NetAppFiles/Properties/Resources.Designer.cs: Language not supported

Comment on lines +148 to +150
### -ProgressAction
{{ Fill ProgressAction Description }}

Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

The -ProgressAction parameter help text is still a placeholder ({{ Fill ProgressAction Description }}), which makes the published cmdlet reference incomplete. Please replace it with an actual description before merging.

Copilot uses AI. Check for mistakes.
Comment on lines +135 to +137
### -ProgressAction
{{ Fill ProgressAction Description }}

Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

The -ProgressAction parameter section uses the placeholder {{ Fill ProgressAction Description }}. Please replace this with real parameter documentation (or regenerate help so the standard ProgressAction description is emitted) to avoid shipping placeholder text.

Copilot uses AI. Check for mistakes.
Comment on lines +71 to +73
### -ProgressAction
{{ Fill ProgressAction Description }}

Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

The -ProgressAction parameter documentation is still the placeholder {{ Fill ProgressAction Description }}. This should be replaced with a meaningful description (usually the standard common-parameter wording) before publishing the help.

Copilot uses AI. Check for mistakes.
@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants