-
-
Notifications
You must be signed in to change notification settings - Fork 951
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
fix bold markup for Atlassian exporter #329
fix bold markup for Atlassian exporter #329
Conversation
Hi @lahma, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution! TTYL, DNFBOT; |
@lahma, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR. |
Hello, @lahma, I'm doing some refactoring in exporters right now. Could you send me the Attlassian spec you use? This will help me not to break your changes. |
@alinasmirnova here's the full markup guide: https://jira.atlassian.com/secure/WikiRendererHelpAction.jspa?section=all These work in both Confluence wiki and JIRA issue tracker. The relevant ones are noformat, tables and text effects (bold). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for another contribution!
buffer.Append(markdownBold); | ||
buffer.Append(line[columnIndex]); | ||
buffer.Append(markdownBold); | ||
buffer.Append(string.Format(boldMarkupFormat, line[columnIndex])); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
@alinasmirnova don't you mind if I merge this PR now? |
dc787d9
to
24dea48
Compare
@adamsitnik thanks for the review, I fixed the usage to use AppendFormat as you suggested. |
Atlassian uses single '*'s around text to make it bold where GitHub uses two. This fixes to allow customization of format for bold text which Atlassian exporter then overrides.