Skip to content

{containerapp} raise ValidationError when value in --yaml is None#6693

Merged
zhoxing-ms merged 6 commits intoAzure:mainfrom
Greedygre:xinyu/fix_yaml_none
Aug 28, 2023
Merged

{containerapp} raise ValidationError when value in --yaml is None#6693
zhoxing-ms merged 6 commits intoAzure:mainfrom
Greedygre:xinyu/fix_yaml_none

Conversation

@Greedygre
Copy link
Copy Markdown
Contributor


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

Related command

az containerapp create/update

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

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.json automatically.
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.

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

azure-client-tools-bot-prd bot commented Aug 28, 2023

️✔️Azure CLI Extensions Breaking Change Test
️✔️Non Breaking Changes

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

Hi @Greedygre,
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in setup.py as well.

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Aug 28, 2023

containerapp

@Greedygre Greedygre force-pushed the xinyu/fix_yaml_none branch from 9554b27 to f005567 Compare August 28, 2023 08:53
@Greedygre
Copy link
Copy Markdown
Contributor Author

Hi @zhoxing-ms
This PR has been reviewed by zunli, could you please help to review and merge? Thanks.

@zhoxing-ms zhoxing-ms merged commit c147e38 into Azure:main Aug 28, 2023
scrappywyrm pushed a commit to scrappywyrm/azure-cli-extensions that referenced this pull request Aug 30, 2023
…ure#6693)

* raise ValidationError when value in --yaml is None

* add test

* raise ValidationError when value in --yaml is None

* add test

* remove usused code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot ContainerApp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants