cketti / ShareIntentBuilder

Share Intent Builder - Android Library for the type-safe creation of Share Intents.
Apache License 2.0
45 stars 8 forks source link

updated dependencies and reduced min sdk to 15 #4

Open kibotu opened 8 years ago

cketti commented 8 years ago

Thank you for trying to contribute. However, there are a couple of issues with this pull request:

Generally, I prefer pull requests that change just one thing, not multiple things at once.

kibotu commented 8 years ago

thanks for the feedback, generelly i just wanted to share the update of all dependencies

minSdkVersion 16 was chosen deliberately because that's the first Android version that properly handles URI permissions with a content:// URI in EXTRA_STREAM

in my opinion this should be a passive requirement not a hard one, because applications lower than 16 couldn't even compile due to manifest merging, even though just this one feature is not supported below 16 for stated reasons

I don't want to use snapshot versions of libraries jitpack is mainly based on tags and less about snapshots, although both are possible

There are some unrelated and unwanted changes (the Jitpack link, commented out android-mvn-push.gradle script)

i prefer to your maven depency but i can't use it since i don't own the credentials, so feel free to re-add it, it's just jitpack couldn't build and i sort of needed this library now. also the library hasn't been updated in quite a while so, forgive me for assuming the pull request might take a while to be considered

other than that, neat library, thanks for sharing, stay awesome~

cketti commented 8 years ago

If you really have to, you can override the library's minSdkVersion. See tools:overrideLibrary-marker.

The snapshot comment was because of the inclusion of Sonatype's snapshot repository and testCompile 'org.robolectric:robolectric:3.2-SNAPSHOT'.