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.

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

wip - migrate to oedipus_lex #2934

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

flavorjones
Copy link
Member

@flavorjones flavorjones commented Jul 19, 2023

What problem is this PR intended to solve?

@zenspider is making the case that oedipus_lex is better, faster, and better supported than rexical. Let's try it (again).

See prior work at #1337

To reproduce the failing test: run bundle exec rake css:clean css:generate

@flavorjones flavorjones force-pushed the flavorjones-migrate-to-oedipus-lex branch from 2d263e0 to b94eda0 Compare July 19, 2023 19:55
@flavorjones flavorjones force-pushed the flavorjones-migrate-to-oedipus-lex branch from b94eda0 to c1f733e Compare July 19, 2023 20:06
@zenspider zenspider force-pushed the flavorjones-migrate-to-oedipus-lex branch 4 times, most recently from d5fb727 to 0df0396 Compare July 20, 2023 03:51
@zenspider zenspider force-pushed the flavorjones-migrate-to-oedipus-lex branch from 0df0396 to 4949437 Compare July 20, 2023 04:19
@zenspider
Copy link
Contributor

All greened up. DO NOT MERGE AS IS... I added a do_parse method to the tokenizer that is bogus... but makes the tests pass??? Needs to be verified.

@flavorjones
Copy link
Member Author

@zenspider Thanks for your work on this. I'm back from vacation and will make time to take a look in the next few days.

@zenspider
Copy link
Contributor

no rush!

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.

None yet

2 participants