Closed baronfel closed 3 years ago
Looks good!
This is green! I want to rebase it into less messy history, but then I think this is ready for review and potential merge.
This looks totally fine. I'll just squash merge if you wish?
Squash merge was basically what I was going to do locally, so that's fine with me.
I believe it's because we're not internalizing the FsLexYacc.Runtime anymore, and instead making it a normal project reference
OK
Done :-)
You have kept the auto-generated FsLex/FsYacc source files @baronfel. Is that intentional?
Woo boy this does a lot of things in service of #143
the aim is to make the .Core packages something that could be embedded in other applications (especially IDE tooling) to make working with fslex/fsyacc files nicer.
I'd expect shortly after these changes to do a pass on the returned data model in the various specs to provide ranges, not just positions, for the structures.