Skip to content

only check for a question of noqanda regex match if output if non-empty#1104

Closed
boegel wants to merge 1 commit intoeasybuilders:developfrom
boegel:run_cmd_qa
Closed

only check for a question of noqanda regex match if output if non-empty#1104
boegel wants to merge 1 commit intoeasybuilders:developfrom
boegel:run_cmd_qa

Conversation

@boegel
Copy link
Member

@boegel boegel commented Dec 9, 2014

... and add debug log message

I ran into issues without this because it took a while before an interactive started spewing output (with OpenFOAM-Extend).

@stdweird: please review

@hpcugentbot
Copy link

Test PASSed.

@stdweird
Copy link
Contributor

stdweird commented Dec 9, 2014

this needs an equivalent fix and unittest in vsc-base run

Copy link
Contributor

Choose a reason for hiding this comment

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

mention the new/updated hitcount in the message

@boegel boegel added this to the v2.x milestone Jan 23, 2016
@akesandgren
Copy link
Contributor

@boegel Relevance?

@akesandgren
Copy link
Contributor

Ping @boegel again

@boegel boegel added the bug fix label Feb 18, 2020
@boegel
Copy link
Member Author

boegel commented Feb 18, 2020

I don't recall this being an issue for a while now, so closing...

@boegel boegel closed this Feb 18, 2020
@boegel boegel deleted the run_cmd_qa branch February 18, 2020 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants