Skip to content

update table formatters#865

Merged
zikalino merged 5 commits intoAzure:masterfrom
gtracer:gt/update_table_formatters
Sep 19, 2019
Merged

update table formatters#865
zikalino merged 5 commits intoAzure:masterfrom
gtracer:gt/update_table_formatters

Conversation

@gtracer
Copy link
Copy Markdown
Contributor

@gtracer gtracer commented Jul 26, 2019


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

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:

@gtracer gtracer force-pushed the gt/update_table_formatters branch from c62a67a to f344549 Compare July 31, 2019 15:46
@gtracer gtracer requested a review from zikalino as a code owner July 31, 2019 15:46
@gtracer gtracer force-pushed the gt/update_table_formatters branch from e64db31 to 69cf247 Compare July 31, 2019 16:13
Copy link
Copy Markdown
Member

@zqingqing1 zqingqing1 left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown

@zikalino zikalino left a comment

Choose a reason for hiding this comment

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

looks good. however we need to produce next release, or it will be included in the wheel when somebody produces next release.

@zikalino zikalino merged commit 5f667f1 into Azure:master Sep 19, 2019
ManuInNZ pushed a commit to ManuInNZ/azure-cli-extensions that referenced this pull request Apr 11, 2020
* update table formatters

* fix linting error and add descriptive error message to multiple agent node pools

* create new whl
openapi-sdkautomation-test bot pushed a commit to openapi-env-test/azure-cli-extensions that referenced this pull request Nov 9, 2020
openapi-sdkautomation-test bot pushed a commit to openapi-env-test/azure-cli-extensions that referenced this pull request Nov 10, 2020
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.

3 participants