glow icon indicating copy to clipboard operation
glow copied to clipboard

Audit glow parser

Open kwannoel opened this issue 3 years ago • 3 comments

In GitLab by @kwanzknoel on Jun 25, 2021, 22:46

Is it in a stable state? Good test coverage, maintainability etc...

Status

  • Very detailed AST for expressions, thanks to @AlexKnauth :slight_smile:.

    I do not think it will be huge maintenance cost, if our program syntax is relatively stable.

  • Unit test coverage low: Only Param, Params, some test cases for curly-braces and simple expressions.

  • Integration tests cover the parsing phase for standard library apps.

  • Error handling status: TODO #198

Recommendations

  • Property-based tests might be a good idea to increase coverage.

kwannoel avatar Sep 22 '21 05:09 kwannoel

In GitLab by @kwanzknoel on Jun 29, 2021, 02:55

changed the description

kwannoel avatar Sep 22 '21 05:09 kwannoel

In GitLab by @AlexKnauth on Jun 29, 2021, 03:05

marked this issue as related to #198

kwannoel avatar Sep 22 '21 05:09 kwannoel

In GitLab by @AlexKnauth on Jun 29, 2021, 03:15

changed the description

kwannoel avatar Sep 22 '21 05:09 kwannoel