-
Notifications
You must be signed in to change notification settings - Fork 47
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
feat: Adding ability to publish to maven central #5
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally LGTM! One thought in-line.
@@ -16,9 +16,17 @@ | |||
|
|||
buildscript { | |||
ext.versions = [ | |||
'android' : [ | |||
"buildTools": "29.0.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any way to pull these from the android-maps-utils Java library? Keeping these in sync will be a pain.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not that I can think of. As long as we keep the dependencies really light (i.e. maps-utils-ktx
should only depend on AMU and kotlin-specific libs) keeping them in sync shouldn't be too much of a hassle.
That said, I'll remove appcompat
as a dependency since it doesn't really need it. Eventually playServicesMaps
and kotlinxCoroutines
should be removed too and instead be a dependency on a maps-ktx
module (i.e. KTX for Maps Core).
# 1.0.0 (2020-04-02) ### Bug Fixes * Add dependencies of maps-utils-ktx in pom. ([#14](#14)) ([d2f14e4](d2f14e4)) * next repositories inside publishing ([38e8dbc](38e8dbc)) * Use api instead of implementation. ([#10](#10)) ([ac40f46](ac40f46)) ### Features * Adding ability to publish to maven central ([#5](#5)) ([5cfa6b1](5cfa6b1)) * Adding KTX to Maps SDK ([#21](#21)) ([3313167](3313167)) * Adding PolyUtil extensions to List<LatLng> ([#16](#16)) ([d4cd6f5](d4cd6f5)) * Create maps-ktx module ([#20](#20)) ([597a70c](597a70c)) ### BREAKING CHANGES * Moved KTX for utils from com.google.maps.android.ktx to com.google.maps.android.ktx.utils to avoid package name conflicts with the new maps-ktx module. * refactor: move core extensions to separate package. * Using shared version. * Adjusting settings.gradle.
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Adding gradle task to publish to maven central.
Relates to #3 🦕