-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Add support to pass name in metadata field of Yaml #8721
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 support to pass name in metadata field of Yaml #8721
Conversation
* Swagger Merge * Fixed Install --------- Co-authored-by: Atharva Udapure <[email protected]>
Co-authored-by: Atharva Udapure <[email protected]>
* Made changes * Added timeout * Added timeout --------- Co-authored-by: Atharva Udapure <[email protected]>
Co-authored-by: Atharva Udapure <[email protected]>
…ows LRO Response Expected SolutionTemplateVersion Data ) (#3) * Fixed The Bug * Nit Changes --------- Co-authored-by: Atharva Udapure <[email protected]>
* Add changes * Some Changes * Merged with main --------- Co-authored-by: Atharva Udapure <[email protected]>
* Merged ConfigTemplate * Merged ConfigTemplate --------- Co-authored-by: Atharva Udapure <[email protected]>
* Some Nit Changes * Fixed uninstall bug * Fixed uninstall bug * Fixed uninstall bug --------- Co-authored-by: Atharva Udapure <[email protected]>
Co-authored-by: Atharva Udapure <[email protected]>
* Fixed the Stuff * remove revision fixed --------- Co-authored-by: Atharva Udapure <[email protected]>
Co-authored-by: Atharva Udapure <[email protected]>
Co-authored-by: Atharva Udapure <[email protected]>
* merge * Merged * Merged * Merged * Add changes --------- Co-authored-by: Atharva Udapure <[email protected]>
* Merged * Merged * Merged * Merged ConfigTemplate * Merged ConfigTemplate --------- Co-authored-by: Atharva Udapure <[email protected]>
* Made-Changes * Made-Changes * Changes * Changes * Removed workflow * Reverted the cli changes * Reverted the cli changes --------- Co-authored-by: Atharva Udapure <[email protected]>
…template creation
|
Validation for Breaking Change Starting...
Thanks for your contribution! |
|
Hi @AbhishekSinha-AEP, |
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
|
The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR. Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions). pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>
|
CodeGen Tools Feedback CollectionThank you for using our CodeGen tool. We value your feedback, and we would like to know how we can improve our product. Please take a few minutes to fill our codegen survey |
|
not the correct repo, closing this |
Added support in Schema, Solution Template and Config Template Resource creation to pass name in Yaml file's metadata object or by either passing in CLI -n argument.
The CLI will now parse the yaml and check if yaml has the metadata.name field with a bunch of checks.
Related command
General Guidelines
azdev style <YOUR_EXT>locally? (pip install azdevrequired)python scripts/ci/test_index.py -qlocally? (pip install wheel==0.30.0required)For new extensions:
About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update
src/index.jsonautomatically.You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify
src/index.json.