Skip to content

Switch parser over to Prism - #543

Open
kddnewton wants to merge 9 commits into
mainfrom
pair-down
Open

Switch parser over to Prism#543
kddnewton wants to merge 9 commits into
mainfrom
pair-down

Conversation

@kddnewton

Copy link
Copy Markdown
Member

No description provided.

@kddnewton
kddnewtonforce-pushed the pair-down branch 2 times, most recently from 080b95e to 016f0c5CompareOctober 29, 2025 22:11
@kddnewton
kddnewtonforce-pushed the pair-down branch 5 times, most recently from 824108e to 8566d42CompareOctober 30, 2025 14:36
@MatthiasPortzel

MatthiasPortzel commented Nov 14, 2025

Copy link
Copy Markdown

Some early results from running on careviso's monolith (I'm obviously excited). Unsurprising but still interesting.

Performance ⚡:
6.3.0:

real	0m5.149s

pair-down:

real	0m1.531s

Some differences don't bother me but I bothered to reduce a couple cases into failing tests.

%
def foo
# put one
puts 1
rescue => err
puts err
end
%
return(
bar(
# foo
foo
)
)
%
def foo
super { a = b if c }
end
%
# rubocop disable
def foo
puts 1
end
# rubocop enable
bar
%
format(
foooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo
).strip
%
bar = [
"foooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo"
].abc.xyz
%
foo
# bar
%

@kddnewton

Copy link
Copy Markdown
MemberAuthor

Thanks @MatthiasPortzel! Yeah I'm still working through the various weird cases. I'll push to this branch once there is more progress.

Sign up for freeto 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.

2 participants

@kddnewton@MatthiasPortzel