Skip to content

Conversation

@Chaoba
Copy link
Contributor

@Chaoba Chaoba commented Mar 22, 2021

No description provided.

@Chaoba Chaoba self-assigned this Mar 22, 2021
@Chaoba Chaoba force-pushed the kl-registry branch 3 times, most recently from ff62b64 to 6fd898e Compare March 23, 2021 03:03
@Chaoba Chaoba requested review from kiryldz, pengdev and tobrun March 23, 2021 10:14
release:
docker:
- image: mbgl/android-ndk-r19:8e91a7ebab
- image: mbgl/android-ndk-r21:769c31f1d8
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably should use ndk-r21e?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's align with maps sdk v9

POM_NAME=Mapbox Maps SDK for Android KTX
POM_DESCRIPTION=A set of Kotlin extensions for the Mapbox Maps SDK for Android.
POM_PACKAGING=aar
REGISTRY_SDK_NAME=mapbox-android-ktx
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this the correct name? looks different than the other names.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems we have never released this before. Please correct me if I'm wrong. @tobrun

Copy link
Member

@pengdev pengdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with 2 questions, please create a ticket to track the tail work of enabling release automation through CI.

@Chaoba Chaoba merged commit 4ccd0a1 into main Mar 23, 2021
@Chaoba Chaoba deleted the kl-registry branch March 23, 2021 13:02
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.

2 participants