Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Measure coverage for integration tests in CI #1893
Measure coverage for integration tests in CI #1893
Changes from 36 commits
99de732
8e59c10
f8c4c20
afb2f7e
fc0fcf7
aac344c
25742ac
481bb13
740486a
9a8e287
1af0efa
ae23bdc
0d7d80f
d3ff0a3
0585c54
89a4a19
aee9d31
ecb7f20
41a524c
356f6b3
0e6ce01
62e831a
7e5ddc3
29c999f
c93efc1
92cf702
d45d96c
b4cd1a3
3f05edf
58e4548
e7fa27a
63fb418
a6cba8e
6325cc0
b9400d4
0c07f87
8519b88
40166bf
9d76d62
4a6a507
e1be4bf
2c5063c
fcf9c93
334ea75
f589a55
8f9d070
c36e85e
2f2b267
4ced21f
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Why are we explicitly checking
matrix.hpc == false
here? If the ghc version is because this is (arbitrarily) our current "distribution" version, won't this break if we then happen to choose the distribution version to be the same one we are generating coverage for? Same question goes for the next 2 steps with the same check. Either something to fix, or something that should have an explanatory comment.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.
The check is here to avoid the problem you identified. If the distribution version is changed to match the HPC version, then this prevents the HPC binaries from clobbering the distribution binaries. For example, assume the distribution is changed to
9.4.4
. The matrix specifies that there are two different Ubuntu builds with GHC 9.4.4. One of those builds has HPC enabled and the other has HPC disabled. This conditional ensures that it's the version with HPC disabled that gets uploaded. I'll add a comment to clarify that.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.
I added a comment clarifying the
matrix.hpc == false
checkThere 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.
Let me know if you still think there's a logic error here
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.
Here (and in the next two steps), why are we concerned with
matrix.run_tests
?In fact, for this entire job, it's not clear to me what
matrix.run_tests
is intended to solve, and why we need it at all in this job. Any ideas?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.
It looks like
matrix.run_tests
exists to avoid uploading build artifacts from theubuntu-20.04
build because none of the testing runs use that version of Ubuntu. Does that sound right to you?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 being said,
run_tests
is implicitly true for the HPC builds so I removed it from the HPC related steps.This file was deleted.