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

CDash Random failure tool patch 2024-02-15 (#600) #605

Conversation

achauphan
Copy link
Collaborator

Related issues

Description

These are requested changes to the Trilinos production version that are being implemented. I've packaged them as a patch here for the TriBITS version.

Let me know if there is a better way to going about these patches going forward. I figured it would be a good idea to include the changes in the production Trilinos version of the script here as well.

Changes

  • Added optional --email-subject-prefix argument to add a prefix string to html page title and email subject line
  • Change test history URL output in singleSummaryReporter to use a hyperlink to clean up output

Report output with multiple random failures can get a bit
hard to read. Condensed the output by morphing test history URL
into a hyperlink.

User story: TRILFRAME-614
Added argument to specify a prefix string for the built html
page title and the email subject. This can help with the
tool's email searchability.
Copy link
Member

@bartlettroscoe bartlettroscoe left a comment

Choose a reason for hiding this comment

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

Changes look great and make perfect sense.

@bartlettroscoe bartlettroscoe merged commit 58c744c into TriBITSPub:master Feb 19, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants