Skip to content

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

Merged
NoriZC merged 9 commits intoAzure:mainfrom
audunn:relase-Microsoft.NetApp-2025-12-01
Mar 16, 2026
Merged

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

Conversation

@audunn
Copy link
Copy Markdown
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
Copy Markdown

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

Copy link
Copy Markdown
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
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@isra-fel
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown
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
Copy Markdown
Member Author

audunn commented Feb 23, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
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
Copy Markdown
Member Author

audunn commented Feb 23, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
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
Copy Markdown
Member Author

audunn commented Feb 23, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
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
Copy Markdown
Member Author

audunn commented Feb 23, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

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

@isra-fel
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@audunn
Copy link
Copy Markdown
Member Author

audunn commented Feb 25, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@isra-fel
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@audunn
Copy link
Copy Markdown
Member Author

audunn commented Mar 5, 2026

@isra-fel can this be merged soon ?

@NoriZC NoriZC self-assigned this Mar 16, 2026
@NoriZC NoriZC merged commit d73ebde into Azure:main Mar 16, 2026
12 checks passed
notyashhh pushed a commit that referenced this pull request Mar 24, 2026
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.

4 participants