Skip to content

add version upgrade#426

Merged
AllyW merged 17 commits into
Azure:devfrom
AllyW:version-upgrade
Jan 8, 2024
Merged

add version upgrade#426
AllyW merged 17 commits into
Azure:devfrom
AllyW:version-upgrade

Conversation

@AllyW
Copy link
Copy Markdown
Contributor

@AllyW AllyW commented Dec 6, 2023

No description provided.

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

Hi @AllyW,
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.

@AllyW AllyW self-assigned this Jan 3, 2024

class VersionUpgradeMod:

def __init__(self, module_name, current_version, is_preview, is_experimental,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is there any reason to keep is_experimental?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is passed from current raw metadata and used to check whether to insert isPreview tag to next extension version for old cli users.

Comment thread azure-cli-diff-tool/azure_cli_diff_tool/version_upgrade.py Outdated
return "".join([str(item) for item in version_arr])


class VersionUpgradeMod:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think version upgrade code should be in azdev instead of azure-cli-diff-tool.

Copy link
Copy Markdown
Contributor Author

@AllyW AllyW Jan 4, 2024

Choose a reason for hiding this comment

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

Version upgrade is working based on diff tool, and in extension ci, version calculation is planned to be used as func call, so packaged with meta-diff together. azdev can expose related cmd later if needed.

Comment thread HISTORY.rst Outdated
Co-authored-by: kai ru <69238381+kairu-ms@users.noreply.github.com>
@AllyW AllyW merged commit 4693ff2 into Azure:dev Jan 8, 2024
RenSilvaAU pushed a commit to RenSilvaAU/azure-cli-dev-tools that referenced this pull request Aug 7, 2025
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