Skip to content

fix backticks in inline code (fixes #312) - #503

Closed
Feder1co5oave wants to merge 2 commits into
markedjs:masterfrom
Feder1co5oave:fix-backticks
Closed

fix backticks in inline code (fixes #312)#503
Feder1co5oave wants to merge 2 commits into
markedjs:masterfrom
Feder1co5oave:fix-backticks

Conversation

@Feder1co5oave

Copy link
Copy Markdown
Contributor

Fixes #312.
Some simple test cases are included in linked issue.

@joshbruce

Copy link
Copy Markdown
Member

Closing as having merge conflicts, stale, tagging with #983

@joshbruce joshbruce closed this Dec 25, 2017
@Feder1co5oave

Copy link
Copy Markdown
Contributor Author

I will add some tests even if it is pointless since whitespace is not checked when testing.

@Feder1co5oave

Copy link
Copy Markdown
Contributor Author

I've added some commits to this branch but they don't show up here, is it because the PR is closed?

@joshbruce

Copy link
Copy Markdown
Member

Don't know. Stupid question time, just to confirm: You did push/sync? (That happened to me when I first realized you could push changes to a PR.)

I can reopen, if you would like. I was going to do it without checking in, but didn't want to get in the way of you testing something based on the "stupid question".

@joshbruce

Copy link
Copy Markdown
Member

Quick search isn't showing me any documentation from GitHub that that block this feature if changes are made to PR that is closed....gotta start heading home. Let me know.

@Feder1co5oave

Copy link
Copy Markdown
Contributor Author

Thing is, I don't see anywhere on this issue page the usual "add commits to branch to add changes to the PR" (something along those lines), so I guess this is not working as I expected but I can't see why.

(yes, I pushed and the new commits showed up on my fork repo, but not here)

Well, I'll open a new pr. I've also rebased so history looks more tidy.

joshbruce added a commit that referenced this pull request Jan 20, 2018
fix backticks in inline code (replaces #503, fixes #312)
zhenalexfan pushed a commit to zhenalexfan/MarkdownHan that referenced this pull request Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants