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

fix bold markup for Atlassian exporter #329

Merged
merged 1 commit into from
Dec 30, 2016

Conversation

lahma
Copy link
Contributor

@lahma lahma commented Dec 18, 2016

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.

@dnfclas
Copy link

dnfclas commented Dec 18, 2016

Hi @lahma, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@dnfclas
Copy link

dnfclas commented Dec 18, 2016

@lahma, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, DNFBOT;

@alinasmirnova
Copy link
Contributor

alinasmirnova commented Dec 19, 2016

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.

@lahma
Copy link
Contributor Author

lahma commented Dec 19, 2016

@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).

Copy link
Member

@adamsitnik adamsitnik left a 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.

@adamsitnik
Copy link
Member

@alinasmirnova don't you mind if I merge this PR now?

@lahma lahma force-pushed the features/atlassian-bold-format branch from dc787d9 to 24dea48 Compare December 26, 2016 06:49
@lahma
Copy link
Contributor Author

lahma commented Dec 26, 2016

@adamsitnik thanks for the review, I fixed the usage to use AppendFormat as you suggested.

@adamsitnik adamsitnik merged commit aabece9 into dotnet:master Dec 30, 2016
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.

None yet

4 participants