-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Fix to show PR coverage report #1812
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
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
|
May this would help you. |
This reverts commit 95e8b1a.
b168ae6 to
558ee3f
Compare
Thank you for your comment! |
ReactiveCircus/android-emulator-runner#385 (comment) Change-Id: Id0dc3b07e9836cb4f3704450e2eb520412ba73b9
Change-Id: Id326aae86bfe9345b1207110bf4e648983c0dd91
558ee3f to
704e79b
Compare
Change-Id: Ic97dffb929cc889c07abe6f4a7938fe1a65ab2e1
Change-Id: Icf249596a5789c1305673cd962744a04d31108c5
Change-Id: Ib0a91c8ce4c032e149b806b14d93ce9a4102987e
Change-Id: Ie6e4712241069da331247c97728bfa8ff72a1dfc
|
All actions are successful, but coverage is not displayed in PR. It is displayed in PR for repository that have been forked. fork repository PR comment
logshttps://github.com/android/nowinandroid/actions/runs/16636920716/job/47079604994#step:12:35 |
What I have done and why
Currently the action to show coverage does not seem to be working.
#1920
Changed the target API to display from 30 to 34.
It looks like API 30 is still turned off in the PR here.
#1724