-
Notifications
You must be signed in to change notification settings - Fork 2.3k
present: support markdown table #219
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
base: master
Are you sure you want to change the base?
Conversation
This PR (HEAD: 7c1df7d) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/tools/+/226208 to see it. Tip: You can toggle comments from me using the |
Message from Rebecca Stambler: Patch Set 1: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Gobot Gobot: Patch Set 1: TryBots beginning. Status page: https://farmer.golang.org/try?commit=bca07cc9 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Gobot Gobot: Patch Set 1: TryBot-Result+1 TryBots are happy. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 1: Run-TryBot+1 (1 comment) +rsc Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
This PR (HEAD: 5fd3dbf) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/tools/+/226208 to see it. Tip: You can toggle comments from me using the |
This PR (HEAD: 166b17f) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/tools/+/226208 to see it. Tip: You can toggle comments from me using the |
This PR (HEAD: 0bbdd35) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/tools/+/226208 to see it. Tip: You can toggle comments from me using the |
Message from peter zhang: Patch Set 5: please review again Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Russ Cox: Patch Set 5: Please add a test in testdata/table.md so we don't break this functionality in the future. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Russ Cox: Patch Set 5: Also if you could please change your Author line to use the full name we have in the CONTRIBUTORS file (Peter Zhang), that would help us when looking through history. git commit --amend --author='Peter Zhang [email protected]' Thanks again. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from peter zhang: Patch Set 5: done Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
This PR (HEAD: e54f61b) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/tools/+/226208 to see it. Tip: You can toggle comments from me using the |
Message from Andrew Bonventre: Patch Set 6: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Gobot Gobot: Patch Set 6: TryBots beginning. Status page: https://farmer.golang.org/try?commit=67e5159e Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Gobot Gobot: Patch Set 6: Build is still in progress... Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 7: Patch Set 6 was rebased Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 7: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Gobot Gobot: Patch Set 7: TryBots beginning. Status page: https://farmer.golang.org/try?commit=314f4c0a Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Gobot Gobot: Patch Set 7: Build is still in progress... Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 7: -Run-TryBot The tests are failing. Please take a look and fix before we can move forward. Thanks. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Russ Cox: Patch Set 5: Also if you could please change your Author line to use the full name we have in the CONTRIBUTORS file (Peter Zhang), that would help us when looking through history. git commit --amend --author='Peter Zhang [email protected]' Thanks again. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from peter zhang: Patch Set 5: done Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 6: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Go Bot: Patch Set 6: TryBots beginning. Status page: https://farmer.golang.org/try?commit=67e5159e Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Go Bot: Patch Set 6: Build is still in progress... Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 7: Patch Set 6 was rebased Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 7: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Go Bot: Patch Set 7: TryBots beginning. Status page: https://farmer.golang.org/try?commit=314f4c0a Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Go Bot: Patch Set 7: Build is still in progress... Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 7: -Run-TryBot The tests are failing. Please take a look and fix before we can move forward. Thanks. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from peter zhang: Patch Set 8: Run-TryBot = +1 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from peter zhang: Patch Set 8: Run-TryBot Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from peter zhang: Patch Set 8: (1 comment)
rebase my branch to origin master, please try again? Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 9: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Go Bot: Patch Set 9: TryBots beginning. Status page: https://farmer.golang.org/try?commit=ed855db4 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Go Bot: Patch Set 9: Build is still in progress... Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 9: -Run-TryBot Have you tried running Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 10: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Go Bot: Patch Set 10: TryBots beginning. Status page: https://farmer.golang.org/try?commit=b543033c Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Go Bot: Patch Set 10: Build is still in progress... Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from peter zhang: Patch Set 10: @andrew,I try to understand parse_test logic Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from peter zhang: Patch Set 10: test passed, please try again Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Andrew Bonventre: Patch Set 11: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Go Bot: Patch Set 11: TryBots beginning. Status page: https://farmer.golang.org/try?commit=80c61e04 Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
Message from Go Bot: Patch Set 11: TryBot-Result+1 TryBots are happy. Please don’t reply on this GitHub thread. Visit golang.org/cl/226208. |
hello, any progress on this PR. I'm looking forward for a solution to add a table to |
No description provided.