Skip to content

[Diagnostics] Add a basic terminal markdown printer for educational notes #28612

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

Merged
merged 1 commit into from
Mar 13, 2020

Conversation

owenv
Copy link
Contributor

@owenv owenv commented Dec 6, 2019

Currently the printer only supports the subset of markdown used by educational notes. It bolds headers, improves the appearance of lists and horizontal rules, and does basic indenting of block quotes and code blocks.

@owenv
Copy link
Contributor Author

owenv commented Dec 6, 2019

I'm not sure who has the most context to review markup library changes these days, it looks like it hasn't changed much in the past few years. If anyone could recommend a reviewer I'd appreciate it!

@owenv
Copy link
Contributor Author

owenv commented Dec 9, 2019

Hi @nkcsgexi , I looked in the code-owners file and saw that you're the swiftMarkup owner. Do you mind taking a look at this PR or recommending a reviewer?

@owenv owenv requested a review from nkcsgexi January 27, 2020 21:01
@owenv owenv force-pushed the terminal_md_viewer branch from c94c9fb to deefec5 Compare January 28, 2020 20:31
@owenv
Copy link
Contributor Author

owenv commented Jan 28, 2020

@swift-ci please test

@owenv
Copy link
Contributor Author

owenv commented Jan 28, 2020

@swift-ci please test windows platform

@owenv
Copy link
Contributor Author

owenv commented Feb 11, 2020

@nkcsgexi friendly ping

@owenv owenv force-pushed the terminal_md_viewer branch from deefec5 to aad09bd Compare March 6, 2020 00:59
@owenv
Copy link
Contributor Author

owenv commented Mar 6, 2020

@swift-ci please test Windows platform

…otes

Currently the printer only supports the subset of markdown used by
educational notes. It bolds headers, improves the appearance of lists
and horizontal rules, and does basic indenting of blockquotes and
code blocks.
@owenv owenv force-pushed the terminal_md_viewer branch from aad09bd to d68089d Compare March 6, 2020 02:04
@owenv
Copy link
Contributor Author

owenv commented Mar 6, 2020

@swift-ci please test Windows platform

@owenv
Copy link
Contributor Author

owenv commented Mar 6, 2020

@owenv
Copy link
Contributor Author

owenv commented Mar 6, 2020

@swift-ci please smoke test Linux

@owenv
Copy link
Contributor Author

owenv commented Mar 9, 2020

@nkcsgexi bump, do you mind looking over these markup changes or recommending a reviewer?

@nkcsgexi
Copy link
Contributor

nkcsgexi commented Mar 9, 2020

@nathawes could you review this PR please?

Copy link
Contributor

@nathawes nathawes left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks @owenv and sorry for the delay in reviewing.

@owenv
Copy link
Contributor Author

owenv commented Mar 12, 2020

@nathawes No problem, thanks for the review!

@owenv
Copy link
Contributor Author

owenv commented Mar 12, 2020

@swift-ci smoke test and merge

1 similar comment
@owenv
Copy link
Contributor Author

owenv commented Mar 13, 2020

@swift-ci smoke test and merge

@swift-ci swift-ci merged commit 3982bb4 into swiftlang:master Mar 13, 2020
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.

4 participants