idaholab / TMAP8

Tritium Migration Analysis Program, Version 8
https://mooseframework.inl.gov/TMAP8/
GNU Lesser General Public License v2.1
15 stars 18 forks source link

Adjust documentation and test coverage checks #100

Closed cticenhour closed 6 months ago

cticenhour commented 6 months ago

Refs #95 Used to test coverage recipes #93

moosebuild commented 6 months ago

Job Documentation on ef4580f wanted to post the following:

View the site here

This comment will be updated on new commits.

moosebuild commented 6 months ago

Job Coverage on ef4580f wanted to post the following:

Coverage

No coverage report(s) were found for the base commit 3ac052eaf4. Full coverage report

This comment will be updated on new commits.

cticenhour commented 6 months ago

Coverage summary from the Test target:

Summary coverage rate:
  lines......: 91.0% (315 of 346 lines)
  functions..: 88.5% (46 of 52 functions)
  branches...: no data found
cticenhour commented 6 months ago

@loganharbour Take a peek and let me know if we need to do anything else here as far as coverage. Some work to do on these test failures after increasing the breadth of our parallel/threads testing, but I will save for later.

cticenhour commented 6 months ago

Requesting a review from @loganharbour confirming that the coverage configuration is fully functional (so that we can close #93), and then a review from @simopier on the content adjustments I made.

loganharbour commented 6 months ago

Looks good: https://mooseframework.inl.gov/tmap8/docs/PRs/100/coverage/

cticenhour commented 6 months ago

Thanks for your help here @loganharbour!