Print out how many commands failed in a multi-command scenario. #52

Manually merged

projectmoon merged 3 commits from really-log-multi-command-failures into master

2021-01-31 14:36:18 +00:00

projectmoon commented

2021-01-30 22:15:07 +00:00

Owner

There was also some minor refactoring to move command response handling into their own functions (one for single response, one for multiple) so that the code is more readable.

The number of failing commands are now printed out when at least one command in a multi-command execution fails. This commit does not introduce printing out WHICH commands failed, nor their error messages. There was also some minor refactoring to move command response handling into their own functions (one for single response, one for multiple) so that the code is more readable.

projectmoon added 1 commit 2021-01-30 22:15:11 +00:00

continuous-integration/drone/push Build is passing Details

continuous-integration/drone/pr Build is passing Details

d0c6ca3de8 Print out how many commands failed in a multi-command scenario.

The number of failing commands are now printed out when at least one
command in a multi-command execution fails. This commit does not
introduce printing out WHICH commands failed, nor their error
messages.

There was also some minor refactoring to move command response
handling into their own functions (one for single response, one for
multiple) so that the code is more readable.

projectmoon added 1 commit 2021-01-31 08:16:27 +00:00

continuous-integration/drone/pr Build is failing Details

continuous-integration/drone/push Build is passing Details

a4e66a0ca6 Basic output for multiple command failures.

Also replace newlines with <br/>s in HTML output.

projectmoon added 1 commit 2021-01-31 14:12:31 +00:00

continuous-integration/drone/pr Build is passing Details

continuous-integration/drone/push Build is passing Details

b3c4d8a38c Centralize plain text formatting at point of message sending.

Instead of relying on all parts of the application to construct both
HTML and plain-text responses, we now construct only HTML responses,
and convert the HTML to plain text right before sending the message to
Matrix.

This is a first iteration, because the plain text has a few extra
newlines than it should, created by use of nested <p> tags.

projectmoon referenced this pull request

2021-01-31 14:13:23 +00:00

Log if there are failures when executing multiple commands #25

projectmoon manually merged commit b3c4d8a38c into master

2021-01-31 14:36:18 +00:00

projectmoon referenced this pull request

2021-01-31 14:37:20 +00:00

Log if there are failures when executing multiple commands #25

No reviewers