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

Fixing badges in README #167

Merged
merged 1 commit into from
Mar 22, 2017
Merged

Conversation

mtlynch
Copy link
Contributor

@mtlynch mtlynch commented Mar 22, 2017

The Markdown for badges in the README has newlines incorrectly separating the
images from the associated URL link, causing the link URLs to show up as text.
This removes the newlines so that the badges render properly.

The Markdown for badges in the README has newlines incorrectly separating the
images from the associated URL link, causing the link URLs to show up as text.
This removes the newlines so that the badges render properly.
@mention-bot
Copy link

@mtlynch, thanks for your PR! By analyzing the history of the files in this pull request, we identified @AndrewSamokhvalov, @Roasbeef and @thofmann to be potential reviewers.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.008%) to 69.026% when pulling 83cf6bb on mtlynch:fix-badges into d958ea0 on lightningnetwork:master.

Copy link
Contributor

@trevinhofmann trevinhofmann left a comment

Choose a reason for hiding this comment

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

ACK

image

* [Contribution guide](https://github.com/lightningnetwork/lnd/blob/master/docs/code_contribution_guidelines.md)
Copy link
Contributor

Choose a reason for hiding this comment

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

?

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, yeah not sure what happened here. Visually, there's no diff at all created by this line.

Copy link
Member

@Roasbeef Roasbeef left a comment

Choose a reason for hiding this comment

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

Excellent! I'd noticed that the formatting starting being rendered incorrectly recently. My guess is that when Github published their formal spec to their flavor of markdown, they modified their parser to comply with the new spec.

Thanks for the review @thofmann! LGTM 🌟

@Roasbeef Roasbeef merged commit c2bdb5a into lightningnetwork:master Mar 22, 2017
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.

5 participants