Markdown Interface Too Simplified

[dsheets]

Since abf78d5 and the switch to omd, the cow markdown interface has been too simple to support existing library users, e.g. opam2web.

Markdown.t is aliased to Html.t which destroys the markdown structure and makes Markdown.to_string lie (or at least hire a lawyer). Because the md AST isn't available, one can't use a Markdown.t to generate a ToC (and the Cow ToC function has now disappeared). To get the basic old features of Cow.Markdown, one now has to add omd as a dependency.

The present design means that Cow can't easily offer multiple HTML serializations of the same markdown which seems rather odd.

Perhaps this was all intended. It just seems odd (and slightly pointless) to me.

[samoht]

Agreed. Cow needs some love, please sends patches!

[avsm]

I have no beef with making this change.

On 11 Feb 2014, at 00:06, Thomas Gazagnaire [email protected] wrote:

Agreed. Cow needs some love, please sends patches!

—
Reply to this email directly or view it on GitHub.

[dsheets]

Whatever change I make will break back compat and increment the version. For now, we will release opam2web 1.3.0 against cow 0.8.1 or earlier. Perhaps I will work on this in a few weeks.

[avsm]

That's fine, we can fix up any existing users. After Feb 27th :)

On 11 Feb 2014, at 00:26, David Sheets [email protected] wrote:

Whatever change I make will break back compat and increment the version. For now, we will release opam2web 1.3.0 against cow 0.8.1 or earlier. Perhaps I will work on this in a few weeks.

—
Reply to this email directly or view it on GitHub.

[mor1]

@dsheets as i think my original commit indicated (and if it didn't, i did on the list and/or irc at the time), this was just a quick hack to get some reasonably standard markdown working as the old cow markdown parser was idiosyncratic (and iirc couldn't handle my pages).

feel free to shred it (either you or anyone else :)