Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

switch to commonmark #1323

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

switch to commonmark #1323

wants to merge 1 commit into from

Conversation

McFoggy
Copy link
Member

@McFoggy McFoggy commented Oct 12, 2016

This PR is work in progress and is only submitted to show the attempt & progress on this subject & to allow discussions.

  • introduce commonmark dependencies
  • use commonmark for markdown rendering
  • move RefLinks into a commonsmark-extension
  • provide better handling/rendering for tasks, options:
    • use an AttributeProvider
    • extend an existing NodeRenderer
    • create an extensions
  • ensure all mardown usage uses Markdown class
  • remove markedj
  • update existing tests, add additionals
  • extend gitbucket to allow additional commonmark extensions
    • via Plugins mechanism
    • via configuration only and class availability in the classpath

Do not hesitate to help on this topic if you are interested in the subject.

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

Successfully merging this pull request may close these issues.

None yet

1 participant