Improve formatting and update deps#41
Open
pugnascotia wants to merge 3 commits into
Open
Conversation
sabarasaba
approved these changes
Jun 1, 2026
Member
sabarasaba
left a comment
sabarasaba
left a comment
There was a problem hiding this comment.
changes lgtm, just one small comment
Also migrate away from yarn to npm
pugnascotia
force-pushed
the
improve-formatting-update-deps
branch
from
83a085c to
0ba98f7
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update all dependencies (except peggy and ts-pegjs) and migrate the ESLint config.
The ESLint config now always requires curly braces. This makes logic branches more obvious.
The PEG libraries haven't been updated because
ts-pegjsis essentially out-of-date, and updating justpeggycreating incompatibility warnings. There is an alternative calledpeggy-ts, but it isn't released to NPM and using it doesn't result in any beneficial changes.