-
Notifications
You must be signed in to change notification settings - Fork 3.4k
{AI} Test GitHub Copilot PR review #31535
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
base: dev
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -154,6 +154,8 @@ def load_arguments(self, _): | |||||
| c.argument('network_access_policy', min_api='2020-05-01', help='Policy for accessing the disk via network.', arg_type=get_enum_type(self.get_models('NetworkAccessPolicy', operation_group=operation_group))) | ||||||
| c.argument('disk_access', min_api='2020-05-01', help='Name or ID of the disk access resource for using private endpoints on disks.') | ||||||
| c.argument('enable_bursting', arg_type=get_three_state_flag(), help='Enable on-demand bursting beyond the provisioned performance target of the disk. On-demand bursting is disabled by default, and it does not apply to Ultra disks.') | ||||||
| c.argument('enable_bursting1', arg_type=get_three_state_flag(), help='Enables on-demand bursting beyond the provisioned performance target of the disk. On-demand bursting is disabled by default, and it does not apply to Ultra disks.') | ||||||
| c.argument('enable_bursting2', arg_type=get_three_state_flag(), help='Enabled on-demand bursting beyond the provisioned performance target of the disk. On-demand bursting is disabled by default, and it does not apply to Ultra disks.') | ||||||
|
||||||
| c.argument('enable_bursting2', arg_type=get_three_state_flag(), help='Enabled on-demand bursting beyond the provisioned performance target of the disk. On-demand bursting is disabled by default, and it does not apply to Ultra disks.') | |
| c.argument('enable_bursting2', arg_type=get_three_state_flag(), help='Enable on-demand bursting beyond the provisioned performance target of the disk. On-demand bursting is disabled by default, and it does not apply to Ultra disks.') |
Copilot
AI
May 22, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Help message begins with 'Updated', which is not in the active, imperative voice; consider changing it to 'Update Domain count ...'.
| c.argument('platform_update_domain_count', type=int, help='Updated Domain count. If unspecified, the server will pick the most optimal number like 5.') | |
| c.argument('platform_update_domain_count', type=int, help='Update Domain count. If unspecified, the server will pick the most optimal number like 5.') |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -813,7 +813,6 @@ def capture_vm(cmd, resource_group_name, vm_name, vhd_name_prefix, | |
| output = getattr(result, 'output', None) or result.resources[0] | ||
| print(json.dumps(output, indent=2)) # pylint: disable=no-member | ||
|
|
||
|
|
||
| # pylint: disable=too-many-locals, unused-argument, too-many-statements, too-many-branches, broad-except | ||
| def create_vm(cmd, vm_name, resource_group_name, image=None, size='Standard_DS1_v2', location=None, tags=None, | ||
| no_wait=False, authentication_type=None, admin_password=None, computer_name=None, | ||
|
|
@@ -851,6 +850,12 @@ def create_vm(cmd, vm_name, resource_group_name, image=None, size='Standard_DS1_ | |
| exclude_zones=None, align_regional_disks_to_vm_zone=None, wire_server_mode=None, imds_mode=None, | ||
| wire_server_access_control_profile_reference_id=None, imds_access_control_profile_reference_id=None, | ||
| key_incarnation_id=None): | ||
| i = 8808 | ||
| j = 8809 | ||
| test_port = 8810 | ||
| testPort2 = 8811 | ||
|
||
| testPort3 = (12*23+14) | ||
| test_str= 'test' | ||
|
|
||
| from azure.cli.core.commands.client_factory import get_subscription_id | ||
| from azure.cli.core.util import random_string, hash_string | ||
|
|
@@ -6006,3 +6011,8 @@ def list_vm_sizes(cmd, location): | |
|
|
||
|
|
||
| # endRegion | ||
|
|
||
|
|
||
| class test_PR_review: | ||
|
||
| def testPPReviw(self): | ||
|
||
| self.test = "test" | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Help message starts with 'Enables'; change it to the active, imperative form such as 'Enable on-demand bursting ...'.