Skip to content

Added changes to CLI Descriptions in paloaltonetworks #9070

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.

Sign up for GitHub

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

Jump to bottom
Merged
necusjz merged 8 commits into from
Aug 20, 2025
Merged

Added changes to CLI Descriptions in paloaltonetworks #9070

necusjz merged 8 commits into from
Aug 20, 2025

Conversation

@jayanthjj
Copy link
Contributor

@jayanthjj jayanthjj commented Aug 20, 2025 •
edited
Loading

This checklist is used to make sure that common guidelines for a pull request are followed.

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
  • My extension version conforms to the Extension version schema

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Copilot AI review requested due to automatic review settings August 20, 2025 05:32
@jayanthjj jayanthjj requested a review from kairu-ms as a code owner August 20, 2025 05:32
@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Aug 20, 2025 •
edited
Loading

️✔️Azure CLI Extensions Breaking Change Test
️✔️Non Breaking Changes

@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Aug 20, 2025

Hi @jayanthjj,
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in setup.py as well.

@yonzhan
Copy link
Collaborator

yonzhan commented Aug 20, 2025

Thank you for your contribution! We will review the pull request and get back to you soon.

@github-actions
Copy link

github-actions bot commented Aug 20, 2025

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

@github-actions
Copy link

github-actions bot commented Aug 20, 2025

CodeGen Tools Feedback Collection

Thank you for using our CodeGen tool. We value your feedback, and we would like to know how we can improve our product. Please take a few minutes to fill our codegen survey

Copilot AI reviewed Aug 20, 2025
View reviewed changes
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

This PR updates CLI command descriptions and docstrings throughout the Palo Alto Networks Azure CLI extension to be more descriptive and user-friendly. The changes replace generic technical references with clear, human-readable descriptions that specify they are for "Palo Alto Networks" resources.

  • Updated all command descriptions to be more descriptive and user-centric
  • Standardized naming to include "Palo Alto Networks" branding consistently
  • Incremented version from 1.1.2b1 to 1.1.3b1 with corresponding HISTORY.rst entry

Reviewed Changes

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

File Description
setup.py Version bump from 1.1.2b1 to 1.1.3b1
HISTORY.rst Added changelog entry for CLI description updates
Multiple command files Updated docstrings and descriptions to be more descriptive and include "Palo Alto Networks" branding

@yonzhan yonzhan requested a review from jsntcy August 20, 2025 05:34
@jayanthjj
Update src/palo-alto-networks/azext_palo_alto_networks/aaz/latest/pal…
25fe9ae 
…o_alto/cloudngfw/firewall/_create.py

Co-authored-by: Copilot <[email protected]>
jayanthjj and others added 3 commits August 20, 2025 11:05
@jayanthjj
Update src/palo-alto-networks/azext_palo_alto_networks/aaz/latest/pal…
0744dda 
…o_alto/cloudngfw/firewall/_delete.py

Co-authored-by: Copilot <[email protected]>
@jayanthjj
Update src/palo-alto-networks/azext_palo_alto_networks/aaz/latest/pal…
b938999 
…o_alto/cloudngfw/firewall/_create.py

Co-authored-by: Copilot <[email protected]>
@jayanthjj
Update src/palo-alto-networks/azext_palo_alto_networks/aaz/latest/pal…
b46c7b0 
…o_alto/cloudngfw/firewall/_delete.py

Co-authored-by: Copilot <[email protected]>
@github-actions
Copy link

github-actions bot commented Aug 20, 2025 •
edited
Loading

@github-actions github-actions bot added the release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. label Aug 20, 2025
necusjz
necusjz reviewed Aug 20, 2025
View reviewed changes
@github-actions github-actions bot removed the release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. label Aug 20, 2025
@necusjz necusjz merged commit c18223d into Azure:main Aug 20, 2025
24 checks passed
@azclibot
Copy link
Collaborator

azclibot commented Aug 20, 2025

[Release] Update index.json for extension [ palo-alto-networks-1.1.2b2 ] : https://dev.azure.com/msazure/One/_build/results?buildId=134431161&view=results

nilo19 pushed a commit to nilo19/azure-cli-extensions that referenced this pull request Aug 27, 2025
@jayanthjj @nilo19
Added changes to CLI Descriptions in paloaltonetworks (Azure#9070)
05934a2 
* Added changes to CLI Descriptions in paloaltonetworks

* Update src/palo-alto-networks/azext_palo_alto_networks/aaz/latest/palo_alto/cloudngfw/firewall/_create.py

Co-authored-by: Copilot <[email protected]>

* Update src/palo-alto-networks/azext_palo_alto_networks/aaz/latest/palo_alto/cloudngfw/firewall/_delete.py

Co-authored-by: Copilot <[email protected]>

* Update src/palo-alto-networks/azext_palo_alto_networks/aaz/latest/palo_alto/cloudngfw/firewall/_create.py

Co-authored-by: Copilot <[email protected]>

* Update src/palo-alto-networks/azext_palo_alto_networks/aaz/latest/palo_alto/cloudngfw/firewall/_delete.py

Co-authored-by: Copilot <[email protected]>

* Update HISTORY.rst

* Update setup.py

* Added fix for integration test in paloalto

---------

Co-authored-by: Jayanth Jayadevan <[email protected]>
Co-authored-by: Copilot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@necusjz necusjz necusjz approved these changes

@kairu-ms kairu-ms Awaiting requested review from kairu-ms kairu-ms is a code owner

@jsntcy jsntcy Awaiting requested review from jsntcy

Assignees

@kairu-ms kairu-ms

Labels

None yet

Projects

None yet

Milestone

September 2025 (2025-09-02)

Development

Successfully merging this pull request may close these issues.

5 participants

@jayanthjj @yonzhan @azclibot @necusjz @kairu-ms