Skip to content

Update default workspace mapping for fairfax#939

Merged
mmyyrroonn merged 3 commits intoAzure:masterfrom
bragi92:kaveesh/updateForFFSupport
Sep 12, 2019
Merged

Update default workspace mapping for fairfax#939
mmyyrroonn merged 3 commits intoAzure:masterfrom
bragi92:kaveesh/updateForFFSupport

Conversation

@bragi92
Copy link
Copy Markdown
Member

@bragi92 bragi92 commented Sep 5, 2019 •
edited
Loading

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
Copy Markdown

azuresdkci commented Sep 5, 2019

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/bragi92/azure-cli-extensions.git@kaveesh/updateForFFSupport#subdirectory=src/$EXT&egg=$EXT"

@bragi92 bragi92 marked this pull request as ready for review September 5, 2019 22:39
@bragi92 bragi92 requested a review from zqingqing1 as a code owner September 5, 2019 22:39
@mmyyrroonn mmyyrroonn merged commit eca53d0 into Azure:master Sep 12, 2019
ManuInNZ pushed a commit to ManuInNZ/azure-cli-extensions that referenced this pull request Apr 11, 2020
@bragi92 @ManuInNZ
Update default workspace mapping for fairfax (Azure#939)
216555a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zqingqing1 zqingqing1 Awaiting requested review from zqingqing1

2 more reviewers

@zikalino zikalino zikalino approved these changes

@mmyyrroonn mmyyrroonn mmyyrroonn approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bragi92 @azuresdkci @zikalino @mmyyrroonn