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 Metalava #5883
Update Metalava #5883
Conversation
scripts/update_metalava.sh
Outdated
# Dev branch has JAVA_HOME defined in the project, if building from development branches, | ||
# uncomment below to exclude JAVA_HOME export and use JAVA_HOME from local machine. | ||
# sed '/^export JAVA_HOME=/d' gradlew > gradlew.temp | ||
# mv gradlew.temp gradlew | ||
# chmod a+x gradlew |
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.
$> make update-metalava
was failing because of ^^ cc @RingerJK @LukasPaczos
scripts/update_metalava.sh
Outdated
|
||
# Pick the release tag you prefer, | ||
# more detailed tags/versions can be found at https://android.googlesource.com/platform/tools/metalava/+refs | ||
git checkout android-12.1.0_r5 |
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.
that what I was looking for 👍
Codecov Report
@@ Coverage Diff @@
## main #5883 +/- ##
=========================================
Coverage 67.55% 67.55%
Complexity 3877 3877
=========================================
Files 591 591
Lines 23844 23844
Branches 2747 2747
=========================================
Hits 16108 16108
Misses 6709 6709
Partials 1027 1027 |
I run into the same issue locally 🤔 I think it has to be with the system version of Java. I'm investigating ⛏️ |
5974d2e
to
e7650cd
Compare
e7650cd
to
68a9625
Compare
After some fighting I was able to update to latest Metalava ( |
Going ahead and merging here 🚀 |
Description
This PR updates Metalava
Closes #5852