Bug 542496 - Add a DocumentBuilder for Creole markup
Summary: Add a DocumentBuilder for Creole markup
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: 3.0.29   Edit
Assignee: Kevin de Vlaming CLA
QA Contact: Leo Dos Santos CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2018-12-06 19:59 EST by Kevin de Vlaming CLA
Modified: 2018-12-17 16:44 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kevin de Vlaming CLA 2018-12-06 19:59:08 EST
WikiText's Creole support is currently limited by the absence of a DocumentBuilder for Creole.

It would be useful to have a DocumentBuilder implementation capable of generating markup in the Creole markup language.
Comment 1 Eclipse Genie CLA 2018-12-06 20:15:33 EST
New Gerrit change created: https://git.eclipse.org/r/133621
Comment 2 Eclipse Genie CLA 2018-12-06 20:16:45 EST
New Gerrit change created: https://git.eclipse.org/r/133621
Comment 3 Leo Dos Santos CLA 2018-12-07 12:55:59 EST
Thank you for providing an implementation Kevin. This contribution is quite large thought, and it would help reviewers and committers if you could break this up into smaller logical contributions. We took a similar approach when I provided the Markdown document builder some time ago https://bugs.eclipse.org/bugs/show_bug.cgi?id=440935. You might want to break this down in a similar manner.

And just to cover all bases, you should confirm that you authored this code and you're within your rights to break down and modify it.
Comment 4 Kevin de Vlaming CLA 2018-12-07 13:01:18 EST
(In reply to Leo Dos Santos from comment #3)
> Thank you for providing an implementation Kevin. This contribution is quite
> large thought, and it would help reviewers and committers if you could break
> this up into smaller logical contributions. We took a similar approach when
> I provided the Markdown document builder some time ago
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=440935. You might want to
> break this down in a similar manner.
> 
> And just to cover all bases, you should confirm that you authored this code
> and you're within your rights to break down and modify it.

Yep, I am the sole author of the code submitted above. Thanks for the advice, I'll modify the review and break it down into smaller commits.
Comment 6 Eclipse Genie CLA 2018-12-07 16:14:42 EST
New Gerrit change created: https://git.eclipse.org/r/133701
Comment 8 Eclipse Genie CLA 2018-12-10 13:00:22 EST
New Gerrit change created: https://git.eclipse.org/r/133795
Comment 10 Eclipse Genie CLA 2018-12-10 16:22:41 EST
New Gerrit change created: https://git.eclipse.org/r/133809
Comment 12 Eclipse Genie CLA 2018-12-10 19:12:52 EST
New Gerrit change created: https://git.eclipse.org/r/133815
Comment 14 Eclipse Genie CLA 2018-12-11 12:14:32 EST
New Gerrit change created: https://git.eclipse.org/r/133878
Comment 17 Leo Dos Santos CLA 2018-12-17 16:33:21 EST
Thanks Kevin! Do you think there's anything else?
Comment 18 Kevin de Vlaming CLA 2018-12-17 16:41:33 EST
(In reply to Leo Dos Santos from comment #17)
> Thanks Kevin! Do you think there's anything else?

I think that covers all of the use cases I could find for a Creole document builder.

(Largely based on what's provided in the spec outlined here: http://www.wikicreole.org/wiki/)
Comment 19 Leo Dos Santos CLA 2018-12-17 16:42:35 EST
Great, I'll close this then. Thank you for the contribution!