grammars-v4 icon indicating copy to clipboard operation
grammars-v4 copied to clipboard

fix golang interpreted string literals

Open uunnii opened this issue 4 years ago • 5 comments
trafficstars

uunnii avatar Nov 06 '21 07:11 uunnii

Could you also provide an example that covers the change?

KvanTTT avatar Nov 06 '21 11:11 KvanTTT

Could you also provide an example that covers the change?

Yes, I added an example in new commits.

uunnii avatar Nov 06 '21 13:11 uunnii

Unfortunately, tests are failing: https://github.com/antlr/grammars-v4/runs/4125916381?check_suite_focus=true#step:4:4431 Could take a look?

KvanTTT avatar Nov 06 '21 17:11 KvanTTT

Unfortunately, tests are failing: https://github.com/antlr/grammars-v4/runs/4125916381?check_suite_focus=true#step:4:4431 Could take a look?

Yes. The logs show that the error message for this test case is different on windows and other platforms, and I added a new commit to fix it.

uunnii avatar Nov 06 '21 20:11 uunnii

@uunnii I see a merge conflict on this PR and some test failures. Are you planning to continue the PR?

teverett avatar Oct 11 '22 14:10 teverett

@uunnii I am closing this for now. You are welcome to re-open it with merge fixes.

teverett avatar Jan 02 '23 21:01 teverett