Skip to content

Avoid printing lines of tildes in catch2#86

Merged
E3SM-Bot merged 1 commit intomasterfrom
bartgol/catch_avoid_tilde_lines
Feb 17, 2021
Merged

Avoid printing lines of tildes in catch2#86
E3SM-Bot merged 1 commit intomasterfrom
bartgol/catch_avoid_tilde_lines

Conversation

@bartgol
Copy link
Copy Markdown
Contributor

@bartgol bartgol commented Feb 17, 2021

Github parses a line of tilde as a code block opening. If tests output is
written in a msg on a PR/issue, this causes parsing mess.

Motivation

When the AutoTester writes a message on a PR, it adds outputs from the tests in case there are failures, but lines of ~ chars mess up the parsing.

Related Issues

Github parses a line of tilde as a code block opening. If tests output is
written in a msg on a PR/issue, this causes parsing mess.
@E3SM-Bot
Copy link
Copy Markdown
Collaborator

Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request.

Copy link
Copy Markdown
Contributor

@jeff-cohere jeff-cohere left a comment

Choose a reason for hiding this comment

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

Approved, subject to successful testing.

@E3SM-Bot
Copy link
Copy Markdown
Collaborator

Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: EKAT_PullRequest_Autotester_Blake

  • Build Num: 115
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
EKAT_SOURCE_BRANCH bartgol/catch_avoid_tilde_lines
EKAT_SOURCE_REPO https://github.com/E3SM-Project/EKAT
EKAT_SOURCE_SHA 8ce2131
EKAT_TARGET_BRANCH master
EKAT_TARGET_REPO https://github.com/E3SM-Project/EKAT
EKAT_TARGET_SHA 2d504e4
PR_LABELS AT: AUTOMERGE;code usability
PULLREQUESTNUM 86
TEST_REPO_ALIAS EKAT

Build Information

Test Name: EKAT_PullRequest_Autotester_Weaver

  • Build Num: 119
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
EKAT_SOURCE_BRANCH bartgol/catch_avoid_tilde_lines
EKAT_SOURCE_REPO https://github.com/E3SM-Project/EKAT
EKAT_SOURCE_SHA 8ce2131
EKAT_TARGET_BRANCH master
EKAT_TARGET_REPO https://github.com/E3SM-Project/EKAT
EKAT_TARGET_SHA 2d504e4
PR_LABELS AT: AUTOMERGE;code usability
PULLREQUESTNUM 86
TEST_REPO_ALIAS EKAT

Build Information

Test Name: EKAT_PullRequest_Autotester_Mappy

  • Build Num: 33
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
EKAT_SOURCE_BRANCH bartgol/catch_avoid_tilde_lines
EKAT_SOURCE_REPO https://github.com/E3SM-Project/EKAT
EKAT_SOURCE_SHA 8ce2131
EKAT_TARGET_BRANCH master
EKAT_TARGET_REPO https://github.com/E3SM-Project/EKAT
EKAT_TARGET_SHA 2d504e4
PR_LABELS AT: AUTOMERGE;code usability
PULLREQUESTNUM 86
TEST_REPO_ALIAS EKAT

Using Repos:

Repo: EKAT (E3SM-Project/EKAT)
  • Branch: bartgol/catch_avoid_tilde_lines
  • SHA: 8ce2131
  • Mode: TEST_REPO

Pull Request Author: bartgol

@E3SM-Bot
Copy link
Copy Markdown
Collaborator

Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED

Pull Request Auto Testing has PASSED (click to expand)

Build Information

Test Name: EKAT_PullRequest_Autotester_Blake

  • Build Num: 115
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
EKAT_SOURCE_BRANCH bartgol/catch_avoid_tilde_lines
EKAT_SOURCE_REPO https://github.com/E3SM-Project/EKAT
EKAT_SOURCE_SHA 8ce2131
EKAT_TARGET_BRANCH master
EKAT_TARGET_REPO https://github.com/E3SM-Project/EKAT
EKAT_TARGET_SHA 2d504e4
PR_LABELS AT: AUTOMERGE;code usability
PULLREQUESTNUM 86
TEST_REPO_ALIAS EKAT

Build Information

Test Name: EKAT_PullRequest_Autotester_Weaver

  • Build Num: 119
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
EKAT_SOURCE_BRANCH bartgol/catch_avoid_tilde_lines
EKAT_SOURCE_REPO https://github.com/E3SM-Project/EKAT
EKAT_SOURCE_SHA 8ce2131
EKAT_TARGET_BRANCH master
EKAT_TARGET_REPO https://github.com/E3SM-Project/EKAT
EKAT_TARGET_SHA 2d504e4
PR_LABELS AT: AUTOMERGE;code usability
PULLREQUESTNUM 86
TEST_REPO_ALIAS EKAT

Build Information

Test Name: EKAT_PullRequest_Autotester_Mappy

  • Build Num: 33
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
EKAT_SOURCE_BRANCH bartgol/catch_avoid_tilde_lines
EKAT_SOURCE_REPO https://github.com/E3SM-Project/EKAT
EKAT_SOURCE_SHA 8ce2131
EKAT_TARGET_BRANCH master
EKAT_TARGET_REPO https://github.com/E3SM-Project/EKAT
EKAT_TARGET_SHA 2d504e4
PR_LABELS AT: AUTOMERGE;code usability
PULLREQUESTNUM 86
TEST_REPO_ALIAS EKAT

@E3SM-Bot
Copy link
Copy Markdown
Collaborator

Status Flag 'Pre-Merge Inspection' - SUCCESS: The last commit to this Pull Request has been INSPECTED AND APPROVED by [ jeff-cohere ]!

@E3SM-Bot
Copy link
Copy Markdown
Collaborator

Status Flag 'Pull Request AutoTester' - Pull Request will be Automerged

@E3SM-Bot E3SM-Bot merged commit b2c150b into master Feb 17, 2021
@E3SM-Bot E3SM-Bot deleted the bartgol/catch_avoid_tilde_lines branch February 17, 2021 18:52
@E3SM-Bot
Copy link
Copy Markdown
Collaborator

Merge on Pull Request# 86: IS A SUCCESS - Pull Request successfully merged

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.

Hack catch2.hpp to avoid char lines that can be parsed as markdown

3 participants