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

Put back coloring context teardown #19227

Merged
merged 1 commit into from
Oct 27, 2021
Merged

Conversation

dschwen
Copy link
Member

@dschwen dschwen commented Oct 27, 2021

Fixes an erroneously removed tear down of the PETSc coloring context.

Refs #19197

@loganharbour loganharbour added the PR: Auto Merge Add this label to have CIVET merge on success label Oct 27, 2021
@moosebuild
Copy link
Contributor

Job Documentation on a394143 wanted to post the following:

View the site here

This comment will be updated on new commits.

@moosebuild
Copy link
Contributor

Job Coverage on a394143 wanted to post the following:

Framework coverage

d536c9 #19227 a39414
Total Total +/- New
Rate 81.79% 81.79% +0.00% 0.00%
Hits 71489 71490 +1 0
Misses 15913 15913 - 1

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

Warnings

  • framework new line coverage rate 0.00% is less than the suggested 90.0%

This comment will be updated on new commits.

@loganharbour loganharbour merged commit 634d737 into idaholab:next Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Auto Merge Add this label to have CIVET merge on success PR: Failed but allowed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants