Skip to content
This repository was archived by the owner on Apr 19, 2018. It is now read-only.

Conversation

Centril
Copy link

@Centril Centril commented Nov 19, 2014

This pull-request accomplishes a few things:

  • It adds groovydocs to all classes in src/main, they can be improved, but I think it's a good balance.
  • It fixes a bug in SdkResolverTest that caused a few tests to fail if on a Windows OS due to \\ vs \\\\.
  • It exposes SdkManagerAction as a standalone component that could potentially be used in a task, or somewhere else... SdkManagerPlugin simply constructs a SdkManagerAction and executes it.
  • Allows non-android projects to use all parts which don't use the android closure.
  • Adds com.jakewharton.sdkmanager as a plugin alias as should be done for newer gradle projects.

Breaking changes? None.
Recommendation for versioning: minor version increment, this adds features and isn't just bugfix.

@Centril
Copy link
Author

Centril commented Dec 31, 2014

Hmmh... had the chance to look at this PR? @JakeWharton

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant