Skip to content
This repository has been archived by the owner on Jan 23, 2024. It is now read-only.

Cleanup of Android API Settings #13

Merged
merged 2 commits into from
May 2, 2017
Merged

Cleanup of Android API Settings #13

merged 2 commits into from
May 2, 2017

Conversation

rarcher
Copy link
Contributor

@rarcher rarcher commented May 2, 2017

Eclipse's insistence on having Android 17 installed was annoying, so I cleaned up the API settings a bit.

More specifically:

  • Updated Eclipse's project.properties file to match the manifest.
  • Removed the minimum and target API settings from the build.gradle file. These are only needed if you want to overwrite the corresponding values in the manifest, and since they are identical to those in the manifest, they are redundant.

Changes have been tested, and seem to work without any problems.

rarcher added 2 commits April 30, 2017 11:54
@rarcher
Update the Eclipse target API to match the one in the manifest.
8a7ccfc
@rarcher
Remove the SDK version attributes from the Gradle script.
3f30101 
These attributes are entirely redundant, as they already exist in the
manifest file. Remove them to reduce the risk of human errors during API
updates.
@jayschwa jayschwa merged commit c6e3499 into jayschwa:master May 2, 2017
@jayschwa
Copy link
Owner

jayschwa commented May 2, 2017

Thanks!

@rarcher
Copy link
Contributor Author

rarcher commented May 2, 2017

No problem. Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rarcher @jayschwa