dlang-community / Pegged

A Parsing Expression Grammar (PEG) module, using the D programming language.
534 stars 66 forks source link

Add generated HTML, for inclusion in the Wiki through rawgit.com. #224

Closed veelo closed 7 years ago

veelo commented 7 years ago

Not sure this'll work, it'd be nice.

PhilippeSigaud commented 7 years ago

I merged it, but I don't think it worked.

veelo commented 7 years ago

It does work now. Link at the very bottom of https://github.com/PhilippeSigaud/Pegged/wiki/Parse-Result.

:-)

PhilippeSigaud commented 7 years ago

Oh, nice! It's fun to play with the tree in that way!

On Thu, Mar 2, 2017 at 7:45 PM, Bastiaan Veelo notifications@github.com wrote:

It does work now. Link at the very bottom of https://github.com/ PhilippeSigaud/Pegged/wiki/Parse-Result.

:-)

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/PhilippeSigaud/Pegged/pull/224#issuecomment-283741921, or mute the thread https://github.com/notifications/unsubscribe-auth/AAjYp2R9m1e107O-OBSX2VHfQaAnCL8eks5rhw48gaJpZM4MQ2SA .

PhilippeSigaud commented 7 years ago

So, I wasn't aware you could generate this HTML5 tree. Maybe I should give some info in the release notes?

On Thu, Mar 2, 2017 at 7:49 PM, Philippe Sigaud philippe.sigaud@gmail.com wrote:

Oh, nice! It's fun to play with the tree in that way!

On Thu, Mar 2, 2017 at 7:45 PM, Bastiaan Veelo notifications@github.com wrote:

It does work now. Link at the very bottom of https://github.com/PhilippeSigaud/Pegged/wiki/Parse-Result.

:-)

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/PhilippeSigaud/Pegged/pull/224#issuecomment-283741921, or mute the thread https://github.com/notifications/unsubscribe-auth/AAjYp2R9m1e107O-OBSX2VHfQaAnCL8eks5rhw48gaJpZM4MQ2SA .

veelo commented 7 years ago

Yes it's very nice to have this example online now, being able to play with it right there is worth much more than a few lines of documentation. Glad you like it. Finding the rule that failed is also much more fun this way. Maybe we need an example of that as well ;-)

A bit of publication in the release notes can do no harm, indeed.

PhilippeSigaud commented 7 years ago

Hmm, since the wiki example was added after the v0.4.0 tag (and I do not know how to move a tag to master), I created a patch release 0.4.1.

On Thu, Mar 2, 2017 at 9:07 PM, Bastiaan Veelo notifications@github.com wrote:

Yes it's very nice to have this example online now, being able to play with it right there is worth much more than a few lines of documentation. Glad you like it. Finding the rule that failed is also much more fun this way. Maybe we need an example of that as well ;-)

A bit of publication in the release notes can do no harm, indeed.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/PhilippeSigaud/Pegged/pull/224#issuecomment-283765079, or mute the thread https://github.com/notifications/unsubscribe-auth/AAjYp3uXBb6coOnTqgAgpnWZf7mR27D8ks5rhyGegaJpZM4MQ2SA .