Skip to content

Conversation

amedama41
Copy link
Contributor

This PR enables user to select detailed commit message style.
The list bellow is the selectable styles.

  • pre: preformatted text (This is same as detailed-message-pre. Now, detailed-message-pre is deprecated).
  • rst: parse messages as reStructuredText and output the result.
  • md: parse messages as Markdown and output the result (This requires recommonmark package).

Adds a new option 'detailed-message-style'.
detailed-message-style lets you select the style of detailed message.
You can select pre (preformatted text) or rst (text parsed as RST).
Examples::

    .. git_changelog::
        :detailed-message-style: pre

    .. git_changelog::
        :detailed-message-style: rst
Adds a new detailed-message-style 'md'.
When you select this style, detailed commit message is parsed as Markdown.
Example:

    .. git_changelog::
        :detailed-message-style: md
Copy link
Owner

@OddBloke OddBloke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @amedama41, this looks really good, thanks!

I have one inline request, and it'd be good if you could add something to CHANGELOG too.

docs/using.rst Outdated
.. note::

The feature to output the messages as Markdown requires recommonmark package.
recommonmark is enable to be installed by pip.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we add an extra feature to setup.py so pip install sphinx-git[markdown] (or something similar) works?

Copy link
Contributor Author

@amedama41 amedama41 Aug 21, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for reviewing.
I have added a markdown extra feature.

@amedama41 amedama41 requested a review from OddBloke November 6, 2019 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants