-
Notifications
You must be signed in to change notification settings - Fork 354
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.
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
added support for other azure cloud environments #437
Conversation
Codecov Report
@@ Coverage Diff @@
## master #437 +/- ##
==========================================
+ Coverage 80.84% 80.91% +0.07%
==========================================
Files 63 63
Lines 6013 6037 +24
==========================================
+ Hits 4861 4885 +24
Misses 760 760
Partials 392 392
|
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.
@kankomi thanks for picking this up! few changes required before we can merge this.
- removed the handling of the arm environment from env.go - use a private property in AzureAuth to get arm environment - updated documentation
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.
Last round and i think we're ready to merge
@kankomi feature is released |
This PR should fix the issues #434 and #391.
This is my first PR so feedback is very welcome.