Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[azdev] Upgrade linter to the public azdev mentioned at hotfix PR #1075 #1082

Merged
merged 3 commits into from
Nov 18, 2019
Merged

[azdev] Upgrade linter to the public azdev mentioned at hotfix PR #1075 #1082

merged 3 commits into from
Nov 18, 2019

Conversation

haroldrandom
Copy link
Contributor

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:

@haroldrandom haroldrandom self-assigned this Nov 8, 2019
@haroldrandom
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@haroldrandom
Copy link
Contributor Author

@fengzhou-msft @zikalino Please help review

@haroldrandom
Copy link
Contributor Author

azdev==0.1.5 has released. This PR can be reviewed.

@zikalino zikalino merged commit 27d4ef5 into Azure:master Nov 18, 2019
@haroldrandom haroldrandom deleted the upgrade_azdev_linter branch November 18, 2019 09:33
@haroldrandom haroldrandom added this to the S161 milestone Nov 18, 2019
ManuInNZ pushed a commit to ManuInNZ/azure-cli-extensions that referenced this pull request Apr 11, 2020
@ManuInNZ
[azdev] Upgrade linter to the public azdev mentioned at hotfix PR Azu…
c89c849 
…re#1075 (Azure#1082)

* [azdev] Upgrade linter to the public azdev mentioned at hotfix PR Azure#1075

* Optimize readability
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zikalino zikalino zikalino approved these changes

@fengzhou-msft fengzhou-msft Awaiting requested review from fengzhou-msft

@Juliehzl Juliehzl Awaiting requested review from Juliehzl

@mmyyrroonn mmyyrroonn Awaiting requested review from mmyyrroonn

@yonzhan yonzhan Awaiting requested review from yonzhan

Assignees

@haroldrandom haroldrandom

Labels
None yet
Projects
None yet
Milestone
S161
Development

Successfully merging this pull request may close these issues.
2 participants
@haroldrandom @zikalino