Skip to content

Conversation

@joelparkerhenderson
Copy link
Contributor

@joelparkerhenderson joelparkerhenderson commented Jan 14, 2022

The purpose of this commit is to help newcomers i.e. developers who are discovering pulldown-cmark for the first time.

This commit has 3 independent items and could easily be split into multiple commits if that's more suitable:

  1. A new example usage code block for "hello world" in the README.

  2. A new program "examples/parser-map-event-print" that uses a parser map in order to print events. This code is similar to the existing "examples/event-filter", plus has a focus on simple printing rather than on content rewriting.

  3. A new program "examples/parser-map-tag-print" that uses a parser map in order to print tags. This code is similar to the existing "examples/event-filter", plus has a focus on start tags including extensions e.g. tables.

@joelparkerhenderson joelparkerhenderson changed the title Add hello world example usage to README Add examples for newcomer developers Jan 15, 2022
@azzamsa
Copy link

azzamsa commented Aug 28, 2022

This is super helpful.

Before looking into this repo, I think the node is limited to Event in pulldown_cmark - Rust. Turns out there is Tag in pulldown_cmark - Rust.

I hope this PR merged soon.

@yankeeinlondon
Copy link

Great to see more examples; this is almost always the fastest way to learn for newcomers. @raphlinus I know you said you're not giving this repo a ton of time but for documentation based enhancements can we push these through? I know you said you were open to others being more involved in the repo ... I would like to help if I can but would probably need a bit of time from you to start to make sure my judgement is sound :)

@marcusklaas
Copy link
Collaborator

Sorry for the long delay on reviewing this.

This looks useful for newcomers to the repo, so I'll merge it. Thanks!

@marcusklaas marcusklaas merged commit cef8cda into pulldown-cmark:master Sep 25, 2022
@lukas-code lukas-code mentioned this pull request May 21, 2023
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