Skip to content
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.

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

Add new command command-change tree-export to export command tree #448

Merged
merged 5 commits into from
May 17, 2024

Conversation

ReaNAiveD
Copy link
Member

Description

I propose to introduce a new command or add new parameters to facilitate the exportation of the command tree.

Once exported, this command tree could be uploaded to our storage account. This would enable the CLI Validation Tool to dynamically load command metadata, significantly improving its functionality and efficiency.

@ReaNAiveD ReaNAiveD requested a review from AllyW May 15, 2024 09:03
Copy link

Hi @ReaNAiveD,
Please write the description of change into HISTORY.rst.
If you want to release a new azdev version, please update the VERSION in init.py as well.

if not module_loader:
continue
module_loader = module_loader[0]
module_path = module_loader.__class__.__module__
Copy link
Member

Choose a reason for hiding this comment

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

extension module name: azext_spring, azext_amcs etc

Copy link
Member Author

Choose a reason for hiding this comment

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

Use _get_command_source to retrieve the extension name.

if not subtree.get(part):
subtree[part] = {}
subtree = subtree[part]
subtree[parts[-1]] = module_name
Copy link
Member

Choose a reason for hiding this comment

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

It would be more clear if expected command tree structure is provided or through test.

@ReaNAiveD ReaNAiveD marked this pull request as ready for review May 16, 2024 03:00
@ReaNAiveD ReaNAiveD merged commit 3b507dc into Azure:dev May 17, 2024
24 checks passed
@ReaNAiveD ReaNAiveD deleted the gen-command-tree branch May 17, 2024 05:50
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