Android - allow changing min sdk version

[PieterVanPoyer]

Platforms affected

Android

Motivation and Context

• Fixes Manifest merger failed : uses-sdk:minSdkVersion 19 cannot be smaller than version 22 declared in library #1070

Give the users the opportunity to overwrite the minSdkVersion without a failing build.

Manifest merger failed : uses-sdk:minSdkVersion 19 cannot be smaller than version 22 declared in
library [:CordovaLib]

Description

This PR uses the overriden cdvMinSdkVersion if it is provided, else it is defaulted to 22 for now.

Testing

I did ran the Java Android tests.
I tested it in next repo: https://github.com/PieterVanPoyer/cordova-plugin-splashscreen-testing-app/tree/testing/changing-min-sdk-version

Checklist

• I've run the tests to see all new and existing tests pass
• I added automated test coverage as appropriate for this change
• Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
• If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
• I've updated the documentation if necessary

[breautek]

Just a few nitpicks.

[codecov-io]

Codecov Report

Merging #1117 (8cf8140) into master (335b0f2) will increase coverage by 0.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1117      +/-   ##
==========================================
+ Coverage   69.71%   70.06%   +0.35%     
==========================================
  Files          20       20              
  Lines        1806     1784      -22     
==========================================
- Hits         1259     1250       -9     
+ Misses        547      534      -13     

Impacted Files	Coverage Δ
bin/templates/cordova/lib/device.js	100.00% <0.00%> (ø)
bin/templates/cordova/lib/build.js	33.33% <0.00%> (+2.22%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 335b0f2...8cf8140. Read the comment docs.

[breautek]

lgtm

[PieterVanPoyer]

@breautek what do you think? Should we wait for another review? Or can this be merged now, or should this be merged later? Kind regards

[PieterVanPoyer]

@timbru31 I completed/resolved your requesting changes. I did learn a bit about Groovy and Gradle with this PR. Nice. If there are still remarks, just let it know.

[timbru31]

LGTM; thanks for addressing my comments.

[breautek]

I'll give it a couple more days or so to give a chance for anybody else to pitch in. Assuming there are no objections I'll merge.

[breautek]

Tests are green, and there haven't been any objections so I'm merging!

Thank you @PieterVanPoyer for your time and effort in preparing this PR and thank you @erisu and @timbru31 for your reviews.