Allow markdown in Intros and Outros as an alternate to plain text

[ruqqq]

Should (at least partly) solve #17.

[ruqqq]

Still need to figure out how to render plain text of the markdowns (including the FreeMarkdown field?)

[codecov-io]

Codecov Report

Merging #31 into master will increase coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
+ Coverage   98.45%   98.54%   +0.08%     
==========================================
  Files           3        3              
  Lines         907      959      +52     
==========================================
+ Hits          893      945      +52     
  Misses          8        8              
  Partials        6        6

Impacted Files	Coverage Δ
hermes.go	77.77% <ø> (ø)	⬆️
flat.go	99.55% <100%> (+0.02%)	⬆️
default.go	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 535ed84...5238a36. Read the comment docs.

[matcornic]

Thanks @ruqqq for the PR. Looks fine to me for now, but could you add/modify some examples and tests ? The goal is checking automatically with CI that Markdown is (and will be - with future changes) displayed as expected.

• tests are in hermes_test.go
• examples are in examples folder. Just go in it and launch go run *.go

To answer your question, plain text version of FreeMarkdown content is generated as follows:

1. Markdown -> HTML (with Blackfriday)
2. HTML -> Text (with htmlToText)

[ruqqq]

Will do so when I get some free time. Curious question: why aren't all the fields markdown enabled by default?

[matcornic]

@ruqqq Hmm, you're right. At first, when I created this issue, I thought I would make all fields Markdown enabled.
But we need to see if themes are robust enough with handling Markdown as well (e.g. Markdown in table)

[danielpoe]

All fields markdown support will be awesome.

[PetitCassoulet]

Hey guys, is there a reason why this still hasn't been merged ? It's a pain to only have plaintext in intros and outros :(

[ChronosMasterOfAllTime]

@PetitCassoulet @ruqqq

I have forked this repo to: https://github.com/go-hermes/hermes

I have implemented this PR here go-hermes#6

[hwebb]

Can this be merged? What are the blocking issues?

[ChronosMasterOfAllTime]

@hwebb I don't think this repo has been maintained for a while. The fork of this repo mentioned in the previous comment implemented your PR. Feel free to make contributions there. I added support for multiple tables, markdown, html strings, custom templates, etc.

[hwebb]

@hwebb I don't think this repo has been maintained for a while. The fork of this repo mentioned in the previous comment implemented your PR. Feel free to make contributions there. I added support for multiple tables, markdown, html strings, custom templates, etc.

Thanks!
I saw it, but I noticed that you didn't merge the new update from this package, so I was wondering if yours was inactive as well.
I'll contribute to it as soon as I can.

[ChronosMasterOfAllTime]

@hwebb It's merged in now; I am still keeping the /v2 for now. But we can always remove it if we feel it's unnecessary