-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
EXP: Test OpenAstronomy/github-actions-workflows PR 199 downstream (Coverage: Upload as separate step) #16419
base: main
Are you sure you want to change the base?
Conversation
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
👋 Thank you for your draft pull request! Do you know that you can use |
I suspect this isn't working correctly due to the significant loss in project coverage, and the many many indirect coverage changes. Going through the logs, this line isn't very encouraging: https://github.com/astropy/astropy/actions/runs/9010438438/job/24756918848#step:3:86 |
We just turned off project coverage for PR. Would it be useful to rebase and try again? Also, can you please elaborate what you see in that line of the log that is not very encouraging? Thanks. |
It says it only uploads one file's worth. It looks like the lack of project coverage could well be a real bug here? |
The line you pointed to still makes no sense to me but I see what you are saying. Lemme see if I can tweak the settings so it sees both files. Good catch!
|
e04d637
to
d181327
Compare
This comment was marked as resolved.
This comment was marked as resolved.
60718a7
to
0a06aa5
Compare
TMP: Remove extra jobs.
0a06aa5
to
606af59
Compare
OK I think I figured it out, @Cadair
|
Description
This pull request is to test OpenAstronomy/github-actions-workflows#199 . This should never be merged.
There is now a new job called
CI / tests / Upload Coverage (pull_request)
.