moose icon indicating copy to clipboard operation
moose copied to clipboard

Added functionality for BoundaryNames in SideSetsFromBoundingBoxGenerator

Open socratesgorilla opened this issue 2 years ago • 4 comments

Closes #22230

socratesgorilla avatar Oct 04 '22 16:10 socratesgorilla

Job Documentation on df97728 wanted to post the following:

View the site here

This comment will be updated on new commits.

moosebuild avatar Oct 04 '22 17:10 moosebuild

These will fail due to unedited mesh_modifier tests and should be reran once the PR deleting them has been dealt with.

socratesgorilla avatar Oct 04 '22 19:10 socratesgorilla

Job Coverage on df97728 wanted to post the following:

Framework coverage

ea3de6 #22293 df9772
Total Total +/- New
Rate 84.29% 84.28% -0.01% 63.16%
Hits 80175 80180 +5 24
Misses 14940 14957 +17 14

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

Warnings

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

This comment will be updated on new commits.

moosebuild avatar Oct 05 '22 00:10 moosebuild

Job Precheck on e22746c wanted to post the following:

Your code requires style changes.

A patch was auto generated and copied here
You can directly apply the patch by running, in the top level of your repository:

curl -s https://mooseframework.inl.gov/docs/PRs/22293/clang_format/style.patch | git apply -v

Alternatively, with your repository up to date and in the top level of your repository:

git clang-format 1cd91e61ff5b6c9d14ef83a94b6ae512e8a3f546

moosebuild avatar Oct 11 '22 19:10 moosebuild

Let's fix the commit history. We dont really want to add then remove a test in the next commit. We dont really want "address review" commit messages either. Each commit should say what it contains

I d squash commit 2 with 5 and 4 with 6 and 7.

In general try to use git commit --fixup as often as possible instead of adding brand new commits

GiudGiud avatar Oct 14 '22 16:10 GiudGiud