ppxlib icon indicating copy to clipboard operation
ppxlib copied to clipboard

Add AST section to examples

Open pedrobslisboa opened this issue 1 year ago • 5 comments

41 files where: 20 are pngs and 8 are old examples files deleted. So don't be scared

Captura de Tela 2024-12-15 às 19 11 58
Captura de Tela 2024-12-15 às 19 12 57

Description

This is the first part of adding the ppx-by-example to ppxlib I will divide it into PR so we can review it better, focusing on the explanation and typos without having tons of files to review. It is really necessary that this section is easy to understand and is aligned with ppxlib. @NathanReb Can you create a ppx-by-example base branch? So I can point this one to it, and after all merges (Other sections) on this branch, we move with the entire ppx-by-example branch to main?

AST

This PR adds the AST sections AST / Building ASTs / Destructing ASTs

How to test it (It's important to check it not only by code but also as styles)

  • At this branch run
     make doc
     open _build/default/_doc/_html/ppxlib/index.html
    
  • Access Examples
  • Access each section:
    • AST
    • Building an AST
    • Destructing an AST

pedrobslisboa avatar Dec 15 '24 21:12 pedrobslisboa

@patricoferris Tagging you for the review as you reacted to this PR

pedrobslisboa avatar Dec 23 '24 15:12 pedrobslisboa

@pedrobslisboa just pushed a ppx-by-example branch, sorry for the delay!

NathanReb avatar Jan 06 '25 15:01 NathanReb

I just updated the ppx-by-example branch and rebased it on top of main.

Can you please rebase this @pedrobslisboa ? I'll review straight away!

NathanReb avatar May 27 '25 08:05 NathanReb

I just updated the ppx-by-example branch and rebased it on top of main.

Can you please rebase this @pedrobslisboa ? I'll review straight away!

Rebased and ready for review.

pedrobslisboa avatar May 28 '25 19:05 pedrobslisboa

🏓

pedrobslisboa avatar Jun 06 '25 14:06 pedrobslisboa