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.

Sign up for GitHub

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

Jump to bottom

Migrate parser from nom #16

Closed
ordian opened this issue Aug 3, 2017 · 0 comments
Closed

Migrate parser from nom #16

ordian opened this issue Aug 3, 2017 · 0 comments
Assignees
@ordian
Labels
C-enhancement Category: Raise on the bar on expectations
Milestone
publish 0.1

Comments

@ordian
Copy link
Member

ordian commented Aug 3, 2017

nom is great for parsing binary formats, but no-so-great for parsing languages and reporting human readable errors. I'm considering combine, pest (procedural branch), peg, larlpop at the moment.
@ordian ordian added the C-enhancement Category: Raise on the bar on expectations label Aug 3, 2017
@ordian ordian added this to the publish 0.1 milestone Aug 3, 2017
@ordian ordian self-assigned this Aug 3, 2017
@ordian ordian mentioned this issue Aug 13, 2017
Merged
bors bot added a commit that referenced this issue Aug 13, 2017
@bors
Merge #26
244fc95 
26: refactor(parser): migrate from nom to combine r=ordian

Closes #16.
@bors bors bot closed this as completed in #26 Aug 13, 2017
epage added a commit to epage/toml_edit that referenced this issue Apr 26, 2024
@epage
Merge pull request toml-rs#16 from epage/renovate/pre-commit-action-3.x
2d5ff55 
chore(deps): Update pre-commit/action action to v3.0.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ordian ordian

Labels
C-enhancement Category: Raise on the bar on expectations
Projects
None yet
Milestone
publish 0.1
Development

No branches or pull requests
1 participant
@ordian