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: 修改语法 hook,完善Cherry Commonmark语法解析 #299

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

stillcalm
Copy link

@stillcalm stillcalm commented Aug 7, 2022

236 通过修改语法 hook,完善Cherry Commonmark,并增加通过10个单测用例(通过 11 个测试,失败 1 个),

(Passed Example 408、483、484、486、494、495、497、499、504、510、596,Failed Example 517)

@sunsonliu sunsonliu added the Event/Tencent Rhino-bird Program 2024 Tencent Rhino-bird Opensource Training Program label Aug 8, 2022
@stillcalm
Copy link
Author

这个pr还能要吗,提交的时候不知道 force-pushed 的危害,然后就提交了。
然后我好奇这个 force-pushed 是什么,这么牛,搜了一下。。。。

@stillcalm
Copy link
Author

stillcalm commented Aug 19, 2022

 // auto.js passed 1
  {
    "markdown": "<MAILTO:[email protected]>\n",
    "html": "<p><a href=\"MAILTO:[email protected]\">MAILTO:[email protected]</a></p>\n",
    "example": 596,
    "start_line": 8792,
    "end_line": 8796,
    "section": "Autolinks"
  }

// emphasis.js passed 1
  {
    "markdown": "*foo *bar**\n",
    "html": "<p><em>foo <em>bar</em></em></p>\n",
    "example": 408,
    "start_line": 6858,
    "end_line": 6862,
    "section": "Emphasis and strong emphasis"
  }

// link.js passed 9,fialed 1(Example 517)
  {
    "markdown": "[](./target.md)\n",
    "html": "<p><a href=\"./target.md\"></a></p>\n",
    "example": 483,
    "start_line": 7544,
    "end_line": 7548,
    "section": "Links"
  },
  {
    "markdown": "[link]()\n",
    "html": "<p><a href=\"\">link</a></p>\n",
    "example": 484,
    "start_line": 7551,
    "end_line": 7555,
    "section": "Links"
  },
  {
    "markdown": "[]()\n",
    "html": "<p><a href=\"\"></a></p>\n",
    "example": 486,
    "start_line": 7565,
    "end_line": 7569,
    "section": "Links"
  },
  {
    "markdown": "[link](\\(foo\\))\n",
    "html": "<p><a href=\"(foo)\">link</a></p>\n",
    "example": 494,
    "start_line": 7637,
    "end_line": 7641,
    "section": "Links"
  },
  {
    "markdown": "[link](foo(and(bar)))\n",
    "html": "<p><a href=\"foo(and(bar))\">link</a></p>\n",
    "example": 495,
    "start_line": 7646,
    "end_line": 7650,
    "section": "Links"
  },
  {
    "markdown": "[link](foo\\(and\\(bar\\))\n",
    "html": "<p><a href=\"foo(and(bar)\">link</a></p>\n",
    "example": 497,
    "start_line": 7662,
    "end_line": 7666,
    "section": "Links"
  },
  {
    "markdown": "[link](foo\\)\\:)\n",
    "html": "<p><a href=\"foo):\">link</a></p>\n",
    "example": 499,
    "start_line": 7679,
    "end_line": 7683,
    "section": "Links"
  },
  {
    "markdown": "[link](/url \"title\")\n[link](/url 'title')\n[link](/url (title))\n",
    "html": "<p><a href=\"/url\" title=\"title\">link</a>\n<a href=\"/url\" title=\"title\">link</a>\n<a href=\"/url\" title=\"title\">link</a></p>\n",
    "example": 504,
    "start_line": 7740,
    "end_line": 7748,
    "section": "Links"
  },
  {
    "markdown": "[link] (/uri)\n",
    "html": "<p>[link] (/uri)</p>\n",
    "example": 510,
    "start_line": 7819,
    "end_line": 7823,
    "section": "Links"
  },
  {
    "markdown": "[foo [bar](/uri)](/uri)\n",
    "html": "<p>[foo <a href=\"/uri\">bar</a>](/uri)</p>\n",
    "example": 517,
    "start_line": 7875,
    "end_line": 7879,
    "section": "Links"
  },

@stillcalm stillcalm changed the title fix: 修改Link hook,完善Cherry Commonmark语法解析 fix: 修改语法 hook,完善Cherry Commonmark语法解析 Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Event/Tencent Rhino-bird Program 2024 Tencent Rhino-bird Opensource Training Program
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants