Updated markdown report for examples#126
Conversation
There was a problem hiding this comment.
The examples aren't coming out in markdown. It's just showing as Given a account balance
|
Fixed the issue you found. Thanks! |
Updated markdown report for examples
There was a problem hiding this comment.
Markdown itself doesn't have html tables and if one wants html reports in pure markdown spec they'll have to write it with html table tags which I think would suck. The alternative solution would be to use GFM which does support markdown but the syntax is different. More info here. I think we should fix this as this isn't proper markdown file anymore.
There was a problem hiding this comment.
I have just done a text table as a code block
Could be a configuration option to do markdown extra which has tables. The end result would be putting the 4 spaces, or not
There was a problem hiding this comment.
Ok. Since I don't know how the GFM actually works I am going to try a few options here. GitHub will show us which one is good:
| sign | action | Result | Errors |
| positive | is | Passed | |
| negative | is not | Failed | Step: Then money <action> dispensed failed with exception|
| sign | action | Result | Errors |
|---|---|---|---|
| positive | is | Passed | |
| negative | is not | Failed | Step: Then money <action> dispensed failed with exception |
| sign | action | Result | Errors |
|---|---|---|---|
| positive | is | Passed | |
| negative | is not | Failed | Step: Then money ; dispensed failed with exception |
There was a problem hiding this comment.
OK. So it seems like we need to add the header separator row with |----|'s and we DO need to escape <>.
There was a problem hiding this comment.
I honestly don't think this needs a configuration options. Both changes are sensible in and out of GitHub or any other viewer with markdown table support.
|
Thanks for this @JakeGinnivan. There were a few issues that I commented on. |
3 participants
No description provided.