Skip to content

Conversation

@Juancpani
Copy link
Contributor

@Juancpani Juancpani commented Oct 3, 2023


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

Related command

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.

@Juancpani Juancpani marked this pull request as ready for review October 4, 2023 01:23
@Juancpani
Copy link
Contributor Author

Can I get reviews on this PR? @Greedygre, @Juliehzl

Copy link

@pagariyaalok pagariyaalok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@Greedygre
Copy link
Contributor

Hi @zhoxing-ms
Could you please help to review this PR? Thanks.

@yanzhudd yanzhudd merged commit 4b3d750 into Azure:main Oct 10, 2023
scrappywyrm pushed a commit to scrappywyrm/azure-cli-extensions that referenced this pull request Oct 16, 2023
Copy link
Contributor

@zhoxing-ms zhoxing-ms Dec 5, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just out of curiosity, I would like to confirm why the projects folder was added at that time? @Juancpani @Greedygre

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Juancpani
Is this projects folder added by mistake?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, the projects folder addition was a mistake. We should remove it.

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.

6 participants