-
Notifications
You must be signed in to change notification settings - Fork 825
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
Move management clients into management package #46
Conversation
This commit moves around the packages for the Azure SDK as discussed in 'management', and storage has been moved to a top level concept.
@@ -1,3 +1,3 @@ | |||
language: go | |||
go: tip | |||
script: go test -v ./azure/... | |||
script: go test -v ./storage/... |
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.
we can add an extra step like go build ./mangement/...
to check if everything at least builds.
LGTM. I can probably split blob storage client to storage/blob/ sometime later. It's not really urgent and requires a bit of extra work. |
@jen20 I really hate Makefiles personally and don't use them except absolutely necessary. The |
@ahmetalpbalkan it's just the easiest most cross platform (Windows excepted, though it's even there) way of running all this stuff prior to checkin. Also, it's using travis rather than circle, but there's still no trivial way to run everything locally with one command to even check builds prior to push. |
@jen20 yep, as you said it requires make on windows :-) I think we should undo that commit and discuss part of another PR. My argument mostly evolves around motivation for "go" command and why they thought Makefile shouldn't exist for Go projects here: https://golang.org/doc/articles/go_command.html |
@ahmetalpbalkan done, for now I've added it to the |
Move management clients into `management` package
This commit moves around the packages for the Azure SDK as discussed in #45.
Management and Storage have been moved to the top level of packages along with Core.