Skip to content

Updated functionality for hack up and added to index#988

Merged
zikalino merged 21 commits intoAzure:masterfrom
GeekTrainer:master
Oct 16, 2019
Merged

Updated functionality for hack up and added to index#988
zikalino merged 21 commits intoAzure:masterfrom
GeekTrainer:master

Conversation

@GeekTrainer
Copy link
Copy Markdown

Updates:

  • Enabled local Azure access to databases
  • Fixed naming issues for MySQL
  • Append random characters for uniqueness
  • Set website as free
  • Added to index.json

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:

@GeekTrainer GeekTrainer requested a review from zikalino as a code owner October 10, 2019 02:17
@azuresdkci
Copy link
Copy Markdown

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

@GeekTrainer
Copy link
Copy Markdown
Author

GeekTrainer commented Oct 10, 2019

@zikalino Any help you can give for merging this would be greatly appreciated! We're hoping to use this at a hack event next week. I'll take care of the linting and other issues first of course :-)

@zikalino zikalino merged commit 564f5a8 into Azure:master Oct 16, 2019
ManuInNZ pushed a commit to ManuInNZ/azure-cli-extensions that referenced this pull request Apr 11, 2020
* Updated functionality for hack up and added to index

* Restructured, fixed linting

* Updated index.json

* Cleaned up hints

* Delete test_hack_scenario.py

* Delete __init__.py

* Delete __init__.py

* fixing linting issues

* Add test

* resolve some index problems

* fix metadata

* change to microsoft just to make sure it's in sync

* removed requires

* another fix

* fix codeowners

* fixed dependencies

* header

* index is not updated
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