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

Align targetSdkVersion in manifests and build script #50

Merged

Conversation

jmoody
Copy link
Contributor

@jmoody jmoody commented Dec 13, 2017

Motivation

Aligns the targetSdkVersion and minSdkVersion across these three sources:

  • server/AndroidManfiest.xml
  • server/instrumentation-backend/AndroidManfest.xml
  • bin/build.sh

The build script now reads the two manifests and errors out if the two version keys are not equal.

@jmoody jmoody added this to the 0.9.2 milestone Dec 13, 2017
@jmoody jmoody self-assigned this Dec 13, 2017
@jmoody jmoody requested a review from john7doe December 13, 2017 15:39
@jmoody jmoody force-pushed the feature/align-sdk-versions-in-manifests-and-build-script branch 2 times, most recently from 9345265 to 07f313d Compare December 14, 2017 07:49
1. server/AndroidManifest.xml

and

2. server/instrumentation-backend/AndroidManifest.xml

now have the same SDK versions
and ensures that the target and min SDK are the same in both
files:

* server/AndroidManifest.xml
* server/instrumentation-backend/AndroidManifest.xml
@jmoody jmoody force-pushed the feature/align-sdk-versions-in-manifests-and-build-script branch from e286784 to 2b526c8 Compare December 14, 2017 08:18
@jmoody jmoody force-pushed the feature/align-sdk-versions-in-manifests-and-build-script branch from 2b526c8 to 4a1f7f1 Compare December 14, 2017 08:55
@jmoody jmoody merged commit 2bd4e33 into develop Dec 14, 2017
@jmoody jmoody deleted the feature/align-sdk-versions-in-manifests-and-build-script branch December 14, 2017 08:59
This was referenced Dec 14, 2017
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.

1 participant