ShareIntentBuilder icon indicating copy to clipboard operation
ShareIntentBuilder copied to clipboard

updated dependencies and reduced min sdk to 15

Open kibotu opened this issue 9 years ago • 3 comments

kibotu avatar Oct 26 '16 13:10 kibotu

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

  • minSdkVersion 16 was chosen deliberately because that's the first Android version that properly handles URI permissions with a content:// URI in EXTRA_STREAM
  • I don't want to use snapshot versions of libraries
  • There are some unrelated and unwanted changes (the Jitpack link, commented out android-mvn-push.gradle script)

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

cketti avatar Oct 26 '16 13:10 cketti

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~

kibotu avatar Oct 26 '16 16:10 kibotu

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'.

cketti avatar Oct 26 '16 16:10 cketti