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

{docs} Add code generation link in developer setup #24513

Merged
merged 2 commits into from
Nov 1, 2022
Merged

Conversation

kairu-ms
Copy link
Contributor

@kairu-ms kairu-ms commented Nov 1, 2022

Add code generation in developer setup

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature


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

Add code generation in developer setup
@kairu-ms kairu-ms self-assigned this Nov 1, 2022
@kairu-ms kairu-ms requested a review from jsntcy November 1, 2022 03:03
@kairu-ms kairu-ms marked this pull request as ready for review November 1, 2022 03:03
@kairu-ms kairu-ms requested a review from jiasli as a code owner November 1, 2022 03:12
@kairu-ms
Copy link
Contributor Author

kairu-ms commented Nov 1, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@kairu-ms kairu-ms requested a review from necusjz November 1, 2022 04:12
@yonzhan
Copy link
Collaborator

yonzhan commented Nov 1, 2022

docs

@yonzhan yonzhan added this to the Nov 2022 (2022-12-06) milestone Nov 1, 2022
@jiasli
Copy link
Member

jiasli commented Nov 1, 2022

CI job Verify src/azure-cli/requirements.py3.Darwin.txt failed:

https://dev.azure.com/azclitools/public/_build/results?buildId=12468&view=logs&jobId=537f1e04-c6d5-53b2-3a1f-f7387cc6e02d&j=537f1e04-c6d5-53b2-3a1f-f7387cc6e02d&t=cae9a690-b596-5b1d-0d0e-9efe890cb6db

python -m azure.cli self-test
ERROR: The command failed with an unexpected error. Here is the traceback:
ERROR: argument _command_package: conflicting subparser: self-test
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py", line 565, in execute
    self.parser.load_command_table(self.commands_loader)
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/azure/cli/core/parser.py", line 100, in load_command_table
    command_parser = subparser.add_parser(command_verb,
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/argparse.py", line 1185, in add_parser
    raise ArgumentError(self, _('conflicting subparser: %s') % name)
argparse.ArgumentError: argument _command_package: conflicting subparser: self-test
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues

I guess it is because it uses Python 3.11 which we don't support yet (#24497).

@jiasli jiasli self-requested a review November 1, 2022 04:28
@jiasli
Copy link
Member

jiasli commented Nov 1, 2022

It's better to have source branch from a forked repo.

@kairu-ms
Copy link
Contributor Author

kairu-ms commented Nov 1, 2022

It's better to have source branch from a forked repo.

I see. I will remove it after merge.

@kairu-ms kairu-ms merged commit fa0d680 into dev Nov 1, 2022
@kairu-ms kairu-ms deleted the kairu-ms-patch-6 branch November 1, 2022 05:51
avgale pushed a commit to avgale/azure-cli that referenced this pull request Aug 24, 2023
* {docs} Add code generation link in developer setup

Add code generation in developer setup

* Update README.md
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.

5 participants