Skip to content
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

Extract some useful classes and simplify Glide.with().using() usage #101

Closed
wants to merge 8 commits into from

Conversation

TWiStErRob
Copy link
Collaborator

The biggest change is (32fccfb):
Make the engine require only the dependencies it will use based on DiskCacheStrategy.

See 0dad499 for the effect it has on usage.

@TWiStErRob TWiStErRob mentioned this pull request Aug 31, 2014
Make library compile/test/checkstyle on Windows (@ignore in comments where needed)
Make the SVG sample a functional app.
Fix version number and format of build.gradle
Remove unintended file and replace with proper scripts.
Review: Remove unnecessary check
@sjudd
Copy link
Collaborator

sjudd commented Sep 2, 2014

LGTM. I'll test and try to merge tonight or tomorrow morning. Thanks!

@sjudd
Copy link
Collaborator

sjudd commented Sep 3, 2014

Rebased onto master starting here: 1e5d51a. Thanks again!

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.

2 participants