Skip to content

Conversation

@weidongxu-microsoft
Copy link
Member

@weidongxu-microsoft weidongxu-microsoft commented Dec 15, 2025

Copilot AI review requested due to automatic review settings December 15, 2025 03:28
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 pull request adapts the Java examples automation tooling to support a new release tag format that includes the Maven group ID prefix. The new format uses com.azure.resourcemanager+azure-resourcemanager-storage_1.0.1-beta.1 while maintaining backward compatibility with the older format azure-resourcemanager-storage_1.0.1-beta.1.

Key Changes:

  • Added a new get_package_name() function to strip the group ID prefix from package names
  • Updated the regex pattern in configuration.json to match both old and new release tag formats
  • Added unit tests for the new package name extraction function

Reviewed changes

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

File Description
tools/azure-rest-api-specs-examples-automation/java/main.py Added get_package_name() function to strip the "com.azure.resourcemanager+" prefix and integrated it into the main workflow to ensure package names are normalized before use
tools/azure-rest-api-specs-examples-automation/java/test_main.py New test file with unit tests for the get_package_name() function covering basic scenarios with and without the group ID prefix
tools/azure-rest-api-specs-examples-automation/automation/configuration.json Updated the regexMatch pattern to make the "com.azure.resourcemanager+" prefix optional, allowing the pattern to match both new and legacy release tag formats

@weidongxu-microsoft
Copy link
Member Author

@weidongxu-microsoft weidongxu-microsoft merged commit d7e3530 into Azure:main Dec 15, 2025
7 checks passed
@weidongxu-microsoft weidongxu-microsoft deleted the examples_automation-adopt-release-tag-change branch December 15, 2025 06:51
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