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

Update readme.md #386

Closed
wants to merge 1 commit into from
Closed

Update readme.md #386

wants to merge 1 commit into from

Conversation

revelt
Copy link

@revelt revelt commented Jan 14, 2019

Let's supplement an example without pipes, so that people could clearly see how to turn AST into MD (string, not HTML), without chained functions. The current example is not very clear and I had to trawl unit tests to find this out.

Let's supplement an example without pipes, so that people could clearly see how to turn AST into MD (string, not HTML), without chained functions. The current example is not very clear and I had to trawl unit tests to find this out.
@ChristianMurphy ChristianMurphy requested a review from a team June 6, 2019 20:10
@wooorm wooorm closed this in a47c3c9 Jul 2, 2019
@wooorm
Copy link
Member

wooorm commented Jul 2, 2019

@revelt Thanks for working on this. I do agree with Christian’s comment above.
All the example uses you found in the unit tests, are documented (and more) in unified’s docs, so instead of this PR I added more links to those.

@revelt
Copy link
Author

revelt commented Jul 2, 2019

Awesome! Thank you @wooorm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants