Skip to content

Conversation

@mmyyrroonn
Copy link
Contributor

@mmyyrroonn mmyyrroonn commented Oct 12, 2019


Fix #936
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:

@azuresdkci
Copy link

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:

docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/MyronFanQiu/azure-cli-extensions.git@fix-monitor-output-table-tsv#subdirectory=src/$EXT&egg=$EXT"

@yonzhan
Copy link
Collaborator

yonzhan commented Oct 13, 2019

@myronfanqiu please take a look.

@zikalino
Copy link

@myronfanqiu Please fix this and it will be good to merge:

C: 25,12: Consider using enumerate instead of iterating with range and len (consider-using-enumerate)

@alexeldeib
Copy link
Contributor

@myronfanqiu thanks for the work, will be happy to get this in 🙂

@mmyyrroonn mmyyrroonn force-pushed the fix-monitor-output-table-tsv branch from 6ec9a44 to b09de88 Compare November 4, 2019 02:34
@mmyyrroonn mmyyrroonn force-pushed the fix-monitor-output-table-tsv branch from ad2b497 to 22477fc Compare November 4, 2019 05:49
@zikalino zikalino merged commit 024ff56 into Azure:master Nov 4, 2019
ManuInNZ pushed a commit to ManuInNZ/azure-cli-extensions that referenced this pull request Apr 11, 2020
* fix table and tsv

* fix tsv and format

* fix

* fix format

* fix style

* update index
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.

tsv and table outputs won't work with az monitor log-analytics query

5 participants