Closed jedbrown closed 1 year ago
Base: 51.04% // Head: 51.05% // Increases project coverage by +0.00%
:tada:
Coverage data is based on head (
0972d4b
) compared to base (36194b9
). Patch coverage: 72.97% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Also, I'm all for clang-format
(though I have noticed that its output doesn't always seem stable between versions).
I think this will all compile with older PETSc too.
Yeah, clang-format changes a little between versions, but it's nice to remove the cognitive load of anything to do with formatting.
We could consider adopting
PetscCall
notation and clang-format if others like it, but this gets things compiling (and fixes a couple format strings).