kcl-lang / tree-sitter-kcl

A tree-sitter grammar for KCL
Apache License 2.0
8 stars 6 forks source link

[PreTest] [LFX] Syntax and test case supplementation for expression nodes #8

Closed Peefy closed 3 months ago

Peefy commented 4 months ago

Enhancement

Syntax and test case supplementation for expression nodes. We need to supplement all expression syntax rules and refine test cases for KCL.

Reference

Grammar: https://github.com/kcl-lang/tree-sitter-kcl/blob/main/grammar.js Tests: https://github.com/kcl-lang/tree-sitter-kcl/tree/main/test/corpus

d4v1d03 commented 4 months ago

Hey @Peefy, could you brief about this issue some more?

Peefy commented 4 months ago

@Vishalk91-4 What problem have you encountered? Do you have any error codes?

Vishalk91-4 commented 4 months ago

@Vishalk91-4 What problem have you encountered? Do you have any error codes?

I mean , I was just trying to add mixin statements, so I added the required module in grammar.js But it couldn't parse, on running the tests, so is there something I need to execute to add them into parser or onto node-types.json

Peefy commented 4 months ago

@Vishalk91-4 What problem have you encountered? Do you have any error codes?

I mean , I was just trying to add mixin statements, so I added the required module in grammar.js But it couldn't parse, on running the tests, so is there something I need to execute to add them into parser or onto node-types.json

@Vishalk91-4 I don't understand the problem. Did you encounter an error while executing tree-site generate or tree-site test? Can you provide more context?

Vishalk91-4 commented 4 months ago

@Vishalk91-4 What problem have you encountered? Do you have any error codes?

I mean , I was just trying to add mixin statements, so I added the required module in grammar.js But it couldn't parse, on running the tests, so is there something I need to execute to add them into parser or onto node-types.json

@Vishalk91-4 I don't understand the problem. Did you encounter an error while executing tree-site generate or tree-site test? Can you provide more context?

There was a tree-sitter-test error I am adding mixin_statement, tried yestarday and today, and that error is fixed now Will raise a PR adding remaining things