scala-parser-combinators icon indicating copy to clipboard operation
scala-parser-combinators copied to clipboard

added a pure functional implementation of operator precedence parsing

Open peteraldous opened this issue 2 years ago • 2 comments

Submitted for discussion; needs tests and documentation.

peteraldous avatar Jun 30 '22 21:06 peteraldous

I don't particularly care for the fact that the Associativity enum lives outside of the Parsers trait, but I don't see a way to include it because of the binary compatibility constraints. It might be there's a simple solution and I'd be happy to find out about it.

peteraldous avatar Mar 08 '23 19:03 peteraldous

I'm still not seeing the same results when I run sbt test locally as the CI pipeline is showing. I've rebased on the new main (as this code is completely orthogonal to the changes there) and addressed the error I saw in the tests. I'll push again if I see more errors. Sorry about the commit spam; I don't know how to reproduce the same errors locally.

peteraldous avatar Apr 05 '23 19:04 peteraldous