mapbox-maps-android
mapbox-maps-android copied to clipboard
Fix revapi release branch
Summary of changes
Fixed revapi check both for release and main branches. Currently it
- fails on PRs to release branch picking incorrect
android-v10.6.0release - the build
$1:
$2:./plugin-compass/build/outputs/aar/plugin-compass-release.aar
$3:compass
$4:android-v10.6.0
Path to previous version of aar is not set, using 10.6.0
aar path is: s3://mapbox-api-downloads-production/v2/mobile-maps-android-compass/releases/android/10.6.0/maven/maps-compass-10.6.0.aar, checking file
- succeedes without any actual comparison on PRs to the main branch - the build
$1:
$2:./plugin-viewport/build/outputs/aar/plugin-viewport-release.aar
$3:viewport
$4:v10.8.0
Path to previous version of aar is not set, using
aar path is: s3://mapbox-api-downloads-production/v2/mobile-maps-android-viewport/releases/android//maven/maps-viewport-.aar, checking file
s3://mapbox-api-downloads-production/v2/mobile-maps-android-viewport/releases/android//maven/maps-viewport-.aar doesn't exist on s3, skipping the check
After the change script correctly compares the previous release, e.g. build with 10.9.0-rc.1
$1:
$2:./plugin-viewport/build/outputs/aar/plugin-viewport-release.aar
$3:viewport
$4:v10.9.0-rc.1
Path to previous version of aar is not set, using 10.9.0-rc.1
aar path is: s3://mapbox-api-downloads-production/v2/mobile-maps-android-viewport/releases/android/10.9.0-rc.1/maven/maps-viewport-10.9.0-rc.1.aar, checking file
Downloading file from s3
User impact (optional)
Pull request checklist:
- [x] Briefly describe the changes in this PR.
- [ ] Include before/after visuals or gifs if this PR includes visual changes.
- [ ] Write tests for all new functionality. If tests were not written, please explain why.
- [ ] Optimize code for java consumption (
@JvmOverloads,@file:JvmName, etc). - [ ] Add example if relevant.
- [ ] Document any changes to public APIs.
- [ ] Run
make update-apito update generated api files, if there's public API changes, otherwise theverify-api-*CI steps might fail. - [x] Update CHANGELOG.md or use the label 'skip changelog', otherwise
check changelogCI step will fail. - [x] If this PR is a
v10.[version]release branch fix / enhancement, merge it tomainfirstly and then port tov10.[version]release branch.
Fixes: < Link to related issues that will be fixed by this pull request, if they exist >
PRs must be submitted under the terms of our Contributor License Agreement CLA.