Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fixed moinpoin parser+merged into master #287

wants to merge 5 commits into


None yet
2 participants

ranft commented Aug 3, 2011

I merged the "moinmoin" branch back into master, and fixed the errors in the MoinMoin parser.

jgm and others added some commits Jan 13, 2011

@jgm jgm Added partial MoinMoin reader, due mostly to Simon Michael. 4b79256
@jgm jgm MoinMoin: changes from Gregory Scott Burd. 0951aff
@jgm jgm Revert "MoinMoin: changes from Gregory Scott Burd."
This reverts commit 0951aff.

The revision did not compile, and used nonexistent type constructors.
@ranft ranft Merge branch 'master' into moinmoin 7d2fe0a
@ranft ranft Fixed the parser.
The changed Code constructor now needs an additional attribute parameter,
which is filled by a nullAttr.

jgm commented Aug 3, 2011

Thank you. I had some difficulties merging, due to the changes in the templates directory, which has become a git submodule. So I am creating a new branch, moinmoin2, with your code changes and some further tweaks of my own.

I am not yet ready to merge this into master; there are too many unimplemented features, and it lacks a test suite. Getting closer, though. I have added a detailed list of features to be added in the comments at the head of the reader.

I'd be okay with leaving some features off in the end - e.g. table parsing - but I'd like to have most of them before including this.


jgm commented Dec 9, 2014

This is bit-rotted, but the work is still in the moinmoin2 branch, if anyone wants it.

@jgm jgm closed this Dec 9, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment