Skip to content
This repository has been archived by the owner on Jun 14, 2024. It is now read-only.

fix white space around links #93

Merged
merged 1 commit into from
Jul 26, 2021

Conversation

kapunahelewong
Copy link
Contributor

@kapunahelewong kapunahelewong commented Jul 26, 2021

This PR is a work in progress trying to fix the extra space that appears around links:

Screen Shot 2021-07-26 at 10 08 47 AM

@kapunahelewong
Copy link
Contributor Author

@kapunahelewong kapunahelewong force-pushed the render-link-fix branch 2 times, most recently from 2d4226e to eaa3b3f Compare July 26, 2021 14:39
Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Make sure that the render-link.html file doesn't end with a newline. That is, the last character of the file must be >.

@kapunahelewong
Copy link
Contributor Author

Oooh that worked perfectly. Thank you @chalin!!

chalin
chalin previously approved these changes Jul 26, 2021
Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

LGTM

@kapunahelewong kapunahelewong marked this pull request as ready for review July 26, 2021 15:23
nate-double-u
nate-double-u previously approved these changes Jul 26, 2021
Copy link
Member

@nate-double-u nate-double-u 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 @kapunahelewong!

@nate-double-u nate-double-u linked an issue Jul 26, 2021 that may be closed by this pull request
Fixes cncf#45

Signed-off-by: Kapunahele Wong <kapunahelewong@gmail.com>
Copy link
Member

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

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

refreshing /lgtm

Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

LGTM (though I'm not sure what your last commit changed relative to earlier).

@nate-double-u
Copy link
Member

LGTM (though I'm not sure what your last commit changed relative to earlier).

I had the same thing over on #86, the GitHub UI said the branch was out of date with main, but I didn't see how. Rebasing and force-pushing seems to have sorted it (but wipes out approvals)

@nate-double-u nate-double-u merged commit f18a002 into cncf:main Jul 26, 2021
@chalin
Copy link
Collaborator

chalin commented Jul 26, 2021

Anything left to do here before this is merged @kapunahelewong? If not, please do the following:

  • Add "Closes Spacing around markdown links #45" to the opening comment if you believe that it closes the issue. (This will allow GitHub to close the issue once this PR is merged.)
  • Go ahead an "squash 'n merge" this PR. (Generally, once you have received the necessary approvals for a PR, feel free to merge it.)

@chalin
Copy link
Collaborator

chalin commented Jul 26, 2021

Oops, @nate-double-u, beat you to it. Well, for next time then :).

@nate-double-u
Copy link
Member

nate-double-u commented Jul 26, 2021

(sorry, I merged it literally the same moment you posted that. I did link the issue before I did so 😅)

@kapunahelewong kapunahelewong deleted the render-link-fix branch August 2, 2021 17:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Spacing around markdown links
3 participants