Skip to content

Change CoverageLineByLine to produce output compatible with that produced by the --coverage command line option#6218

Merged
fingolfin merged 1 commit intomasterfrom
mh/CoverageLineByLine
Feb 9, 2026
Merged

Change CoverageLineByLine to produce output compatible with that produced by the --coverage command line option#6218
fingolfin merged 1 commit intomasterfrom
mh/CoverageLineByLine

Conversation

@fingolfin
Copy link
Member

That is, when asking the profiling package to process multiple files produced by each of the two options, this should just work, while currently it leads to this error:

Error, Some profiles use wall time, some use CPU time

Let it produce output compatible with the `--coverage` command
line option.
@fingolfin fingolfin added kind: bug Issues describing general bugs, and PRs fixing them topic: library release notes: use title For PRs: the title of this PR is suitable for direct use in the release notes labels Jan 31, 2026
@ThomasBreuer ThomasBreuer changed the title Change CoverageLineByLine to produce output compatible with that produced by the `--coverage command line option Change CoverageLineByLine to produce output compatible with that produced by the --coverage command line option Feb 2, 2026
@fingolfin
Copy link
Member Author

@ChrisJefferson approved in a private slack chat but can't currently login to GitHub.

@fingolfin fingolfin merged commit f6c2800 into master Feb 9, 2026
32 checks passed
@fingolfin fingolfin deleted the mh/CoverageLineByLine branch February 9, 2026 12:14
cdwensley pushed a commit to cdwensley/gap that referenced this pull request Feb 26, 2026
Let it produce output compatible with the `--coverage` command
line option.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-to-4.15 kind: bug Issues describing general bugs, and PRs fixing them release notes: use title For PRs: the title of this PR is suitable for direct use in the release notes topic: library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant