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

Change MooseDocs CIVET mergeresults command defaults and add extension documentation #29566

Open
wants to merge 3 commits into
base: next
Choose a base branch
from

Conversation

cticenhour
Copy link
Member

Closes #29565

@gambka I discovered this while testing with your BISON branch. This should make the output of !civet mergeresults consistent for the release versions of BISON docs (or any older version of the docs built locally) by default.

@cticenhour cticenhour self-assigned this Dec 17, 2024
@moosebuild
Copy link
Contributor

moosebuild commented Dec 18, 2024

Job Documentation, step Docs: sync website on e6b16ef wanted to post the following:

View the site here

This comment will be updated on new commits.

@moosebuild
Copy link
Contributor

moosebuild commented Dec 18, 2024

Job Coverage, step Generate coverage on e6b16ef wanted to post the following:

Framework coverage

Coverage did not change

Modules coverage

Coverage did not change

Full coverage reports

Reports

This comment will be updated on new commits.

@moosebuild
Copy link
Contributor

Job Coverage, step Verify coverage on 1b0401f wanted to post the following:

The following coverage requirement(s) failed:

  • Failed to generate level_set coverage rate (required: 85.0%)
  • Failed to generate peridynamics coverage rate (required: 77.0%)

cticenhour added a commit to cticenhour/mastodon that referenced this pull request Dec 18, 2024
MooseDocs.extensions.civet is not active in Mastodon's config.yml

Refs idaholab/moose#29566
cticenhour added a commit to cticenhour/mastodon that referenced this pull request Dec 19, 2024
MooseDocs.extensions.civet is not active in Mastodon's config.yml

Refs idaholab/moose#29566
cticenhour added a commit to cticenhour/mastodon that referenced this pull request Dec 19, 2024
MooseDocs.extensions.civet is not active in Mastodon's config.yml

Refs idaholab/moose#29566
@moosebuild
Copy link
Contributor

Job Documentation controlled apps on e6b16ef : invalidated by @cticenhour

Griffin and Sockeye have been patched.

@moosebuild
Copy link
Contributor

Job Documentation public apps on e6b16ef : invalidated by @cticenhour

Mastodon has been patched.

The command will now always look to the current git commit SHA to determine the linked CI results by default. The old behavior is available using use_current_hash=False in the command.

This commit also adds documentation for MooseDocs CIVET extension

Refs idaholab#29565
Some apps do not have this extension enabled
Copy link
Member

@loganharbour loganharbour left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unit tests an option?

@cticenhour
Copy link
Member Author

@loganharbour Yeah, I can add one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MooseDocs CIVET extension mergeresults command always shows most up-to-date hashes
3 participants