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
Lower minimum dependency #14
Conversation
Some of the other minor changes were just from the code style checker warning me of violations that needed to be fixed before the project would |
Thanks! I always planned on having this work on all versions. I will take a look, and get this merged in when I get a chance. |
Might be nice to integrate this with https://github.com/JakeWharton/sdk-manager-plugin, so that the Android-20 (or whatever version you eventually build against) will be automatically downloaded if its not already present in the builder's Android SDK directory (or, you know, Google could just do the sane thing and actually include the SDKs in MavenCentral 😁 A man can dream, can't he?) |
In order to integrate with that I think the main lib would need to be moved to an aar, which is what I was thinking anyways. I'll be super busy and possibly with out internet for a few days, so as soon as I get a chance I'll check that out and get this PR merged in. And yes, every day I hope Google will start putting android libraries into central. Maybe one day... |
I finally found a 2.3 device to test on. All the samples and tests actually run on it. This is a bug I introduced where the variables in the ext closure are never actually used. I just fixed that. Can you provide more information on how you are getting |
Just pushed the latest changes to artifactory. Can you see if they help. repositories {
jcenter()
maven {
url = 'https://oss.jfrog.org/oss-snapshot-local/'
}
}
...
dependencies {
compile 'com.andrewreitz:shillelagh:0.4.0-SNAPSHOT'
provided 'com.andrewreitz:shillelagh-processor:0.4.0-SNAPSHOT'
...
} |
Yup, you're right, works fine for me. Maybe I was doing something strange with building or including the library on my end. Thanks for looking into this! Will close this PR now. |
The snapshot ended up solving it? |
Also, sorry, I did a rebase which screwed up giving you credit for the checkstyle fixes. |
Yup, snapshot is working fine. No problem about the checkstyles :) Thanks again! |
Great! I was having issues with the samples running in the snapshot now :) I need to figure out a balance between the two... |
Right now you are manually including API Level 15, but most of the functionality you're relying on has been present in Android since before that (at least up until API Level 10, where RXJava is supported). Previously, you would receive an error message like this if attempting to use this library on an earlier version;
After this change, I was able to build a project and run it on an Android 2.3.7 device properly.
Keep up the great work! This library shows a ton of promise.