-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Adding ruleworks grammar. #4097
Conversation
Any idea what this failure is about? Test.ts:199:25 - error TS1003: Identifier expected. 199 const tree = parser.(); I am slowly getting a build/test environment setup on my system and I get the same result. I don't see anything that tells what/where the problem is located. bill |
@whcox603 i had this exact issue last week. It needs an EOF. Try changing this to
|
I'll fix the tester to give a betyer warning. |
Much better. I updated ruleworks.g4 |
It looks like the 3 failures are with the Go target, but not much to go on (pun unintended.) On my system the Go test succeeded. I fixed the conflict with action (renamed to action_). |
Bumps [github.com/antlr4-go/antlr/v4](https://github.com/antlr4-go/antlr) from 4.13.0 to 4.13.1. - [Release notes](https://github.com/antlr4-go/antlr/releases) - [Commits](antlr4-go/antlr@v4.13.0...v4.13.1) --- updated-dependencies: - dependency-name: github.com/antlr4-go/antlr/v4 dependency-type: indirect update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Dependabot updated the go.mod file to remove the reference to Antlr. So, the code that was there to yank out the version doesn't find one. (The code should have checked for a missing reference so it doesn't try to then do a bogus "go get".) I updated the Go templates to put the version directly in the build.* scripts. Please sync your PR with master. |
Much better. Thanks to you both (@teverett @kaby76) for the assistance. The coverage is not that great (around 50%), but there were only 3 sample programs provided. I have built the compiler and I will use that to test out some more sample programs. One problem is that the SQL calls only work with VAX rdb, which I don't have access to. |
What a great contribution. Thanks @whcox603 . |
No description provided.