This repository has been archived by the owner on Feb 3, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 66
Feature: getting started script #475
Merged
Merged
Changes from 28 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
ca499b2
initial changes for getting started scripts
paselem 62c1518
add temp error handling
paselem 4754ca3
rename file - fix typo
paselem 3f05898
add debug strings
paselem 9bac57f
add handling for existing user
paselem 9ab8810
WIP: wait for subprocess to complete to get exit code
paselem a7076bf
WIP: handle existing user and refactor code
paselem d93e73e
WIP: add missing return statements
paselem 8c80ba0
WIP: fix typo
paselem e6b7d53
start sdk refactor
jafreck 23ff3e2
mostly working create
jafreck 2576393
working happy create path
jafreck 010b8c1
handle errors for vnet, aad application
jafreck e33e0c8
make account setup interactive
jafreck 746e7d0
add prompt
jafreck 51997e6
add docs
jafreck d34b05a
rename account_setup_refac to account_setup
jafreck 742dc6b
Merge branch 'master' into feature/getting_started_scripts
jafreck d4c261a
add some logging
jafreck c7cf1dc
Merge branch 'feature/getting_started_scripts' of github.com:azure/az…
jafreck 1049419
pip install msrest, azure-cli-core, import issues
jafreck b59848b
remove in script pip, add shell wrapper program
jafreck 96dd518
ellipsis to period
jafreck 29b4111
update branch name for account_setup.sh
jafreck 35c5ee9
docstring
jafreck 3851dd4
retry resource group creation
jafreck 56f1919
fix typo, update retry
jafreck 3616f0a
explicitly set output location
jafreck a6cbd56
wget overwrite flag, docs update
jafreck 84a4bc2
add prompt for multi tenants
jafreck 4ebc4f3
fix bug with batch account creation
jafreck 75b039e
Merge branch 'master' into feature/getting_started_scripts
jafreck 60bdc18
Merge branch 'master' into feature/getting_started_scripts
jafreck 30fb1ae
Merge branch 'master' into feature/getting_started_scripts
jafreck 76e95f5
add spinner, print statements, fix formatting bug
jafreck d0c7a41
Merge branch 'master' into feature/getting_started_scripts
jafreck 27e45d8
fix param bug
jafreck File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we comfortable enough to just pip install? Technically it should be the same thing, right??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
technically, yes. But I elected not to make that change here until our release on pip is "official". I can do it now if you'd prefer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it just seems so much easier now... even if the version shows up as '0.6.1b' for example...