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

Update Default SDK Version to API 23 #733

Merged

Conversation

caseykulm
Copy link
Contributor

In a while this will be able to be changed to API 24, but for now it could definitely stand to be updated to API 23. I'm also not sure if there is any good place (or if it would be desirable) to log in the build process that the android-maven-plugin has decided to choose a default api level, and maybe a reference to the pom option to configure it.

@malachid
Copy link
Contributor

Actually, 24 is already available for using in published apps.

@caseykulm
Copy link
Contributor Author

Yeah, this could be bumped up to api 24. Wasn't sure whether that should happen yet since it's not technically available for any devices yet. Maybe this pr could just be closed out, and redone in a couple weeks/months when Android N has a full release? or it could be updated here now, but it seems a bit early at the moment.

@mosabua
Copy link
Member

mosabua commented Jun 27, 2016

Can you add a changelog update? Then I can merge.

@caseykulm caseykulm force-pushed the update_default_sdk_version_23 branch from 5b5311d to c393ec4 Compare June 27, 2016 19:56
@caseykulm
Copy link
Contributor Author

Rebased to one commit.

@caseykulm caseykulm force-pushed the update_default_sdk_version_23 branch from c393ec4 to d8b8bcf Compare June 29, 2016 15:14
@caseykulm
Copy link
Contributor Author

Rebased again on latest master, as well as fixing my changelog format to match the other changelog entries.

@mosabua
Copy link
Member

mosabua commented Jun 29, 2016

Nice!

@mosabua mosabua merged commit c1971e4 into simpligility:master Jun 29, 2016
@caseykulm caseykulm deleted the update_default_sdk_version_23 branch June 29, 2016 18:09
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.

3 participants