Skip to content
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

Meta-issue: current issues with the coverage job #188

Closed
5 tasks done
DilumAluthge opened this issue Jul 31, 2022 · 11 comments
Closed
5 tasks done

Meta-issue: current issues with the coverage job #188

DilumAluthge opened this issue Jul 31, 2022 · 11 comments

Comments

@DilumAluthge
Copy link
Member

DilumAluthge commented Jul 31, 2022

This is a great step! But there seem to be a few issues still, looking at https://app.codecov.io/gh/JuliaLang/julia

Originally posted by @vtjnash in #136 (comment)

@IanButterworth
Copy link
Contributor

Are coverage files actually uploaded for PRs? None of these seem to be? https://app.codecov.io/gh/JuliaLang/julia/pulls

I want to see if JuliaLang/julia#46403 works

@DilumAluthge
Copy link
Member Author

We do not upload coverage on PRs, because coverage takes 3-4 hours to run.

You can make a PR to this repo, and in the PR, you set it to use your custom branch of Julia instead of the master branch of Julia. I don't actually remember where you specify your custom branch name, but @staticfloat or @brenhinkeller will know.

@IanButterworth
Copy link
Contributor

Makes sense! This is me trying. Although I think it might hit a build bug on non-macos that I can't figure out..
#194

@brenhinkeller
Copy link
Contributor

brenhinkeller commented Aug 23, 2022

Sounds like you may have figured it out but FWIW re @dilumalthuge's point IIRC one way to pick a custom branch might be to set UPSTREAM_BRANCH as an environment variable: https://github.com/JuliaCI/julia-buildkite/blob/main/.buildkite/hooks/post-checkout

@vtjnash
Copy link
Member

vtjnash commented Sep 9, 2022

Here is the old script that addressed most of these issues in buildbot https://github.com/JuliaCI/julia-buildbot/blob/96f24ff3fcfff05c5411662363483271d364ee59/master/coverage.py#L23-L24

@vtjnash
Copy link
Member

vtjnash commented Jan 16, 2023

bump?

@kshyatt
Copy link
Contributor

kshyatt commented Jan 31, 2023

Items 1 and 3 have been fixed.

@kshyatt
Copy link
Contributor

kshyatt commented Jan 31, 2023

2 and 4 I just confirmed are still problems.

@kshyatt
Copy link
Contributor

kshyatt commented Feb 4, 2023

Looking at https://app.codecov.io/gh/JuliaLang/julia/commit/c4fd8a48383235e818c44231d7507168c27f393b/tree/stdlib, 4 seems to have been fixed.

@kshyatt
Copy link
Contributor

kshyatt commented Feb 6, 2023

https://codecov.com/gh/julialang/julia/tree/2a9f4412a00cbaf366895083f651d5d7906cce1f the extraneous file is also gone

@kshyatt
Copy link
Contributor

kshyatt commented Feb 7, 2023

Coverage dropped back down to ~89%. Looks like we can close this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants