Skip to content

Remove chumsky crate#849

Draft
vinc wants to merge 9 commits into
trunkfrom
chore/remove-chumsky
Draft

Remove chumsky crate#849
vinc wants to merge 9 commits into
trunkfrom
chore/remove-chumsky

Conversation

@vinc
Copy link
Copy Markdown
Owner

@vinc vinc commented May 13, 2026

  • Rewrite INI parser with Nom
  • Rewrite Brainfuck parser with Nom

@vinc
Copy link
Copy Markdown
Owner Author

vinc commented May 15, 2026

Last year I tried to replace nom with chumsky for all my parsing needs.

I started by implementing a small Brainfuck interpreter in #750 then a INI config parser in #759 but for the spell command it turned out to be easier to write a hand parser than using chumsky.

I realized that I preferred nom better to keep the complexity low by combining many small functions. I should probably look into nom_locate to get back what I'll miss from chumksy but it turns out that I can do more than what I though to locate errors out of the box.

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.

1 participant