-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Fix to show PR coverage report #1812
base: main
Are you sure you want to change the base?
Conversation
Change-Id: I0fcb548c30c0cd8fce47fe70e086d43e90813e8d
b2e624e
to
c9ed908
Compare
Hmm.
Of course, locally,
|
It is as though need to be re-run CI. |
Change-Id: Ie5f686baa54cfaeea96a76c366a6ce464a9659db
I did an empty commit and re-run, but it failed as before 🤔 |
The error
Could come from resource limitation problem. Also, now there is some problems with ubuntu-latest as they are migrating it to ubuntu-24.04, can you retry with ubuntu-22.04 or ubuntu-20.04 instead? |
Change-Id: I09ed78d4aee0492497a71bd697a23843537a7ca7
Also, could be related to #1738 |
https://github.com/android/nowinandroid/actions/runs/12885712069/job/35924748312#step:9:685 |
Change-Id: Iece2fd3f9a589e7d87eff6367b2bc7e65898d366
What I have done and why
Currently the action to show coverage does not seem to be working.
#1803
Changed the target API to display from 30 to 34.
It looks like API 30 is still turned off in the PR here.
#1724