JSAbrahams / mamba

🐍 The Mamba programming language, because we care about safety
MIT License
85 stars 3 forks source link

Filter comments when parsing #381

Closed JSAbrahams closed 1 year ago

JSAbrahams commented 1 year ago

Summary

This greatly simplifies (potential) parse logic as we needn't deal with possible comment locations. We now accept that comments are not preserved in the output, however.

codecov[bot] commented 1 year ago

Codecov Report

Merging #381 (7705e83) into develop (63cc218) will decrease coverage by 0.04%. The diff coverage is 85.18%.

@@             Coverage Diff             @@
##           develop     #381      +/-   ##
===========================================
- Coverage    87.00%   86.96%   -0.05%     
===========================================
  Files          109      109              
  Lines        11950    11950              
===========================================
- Hits         10397    10392       -5     
- Misses        1553     1558       +5     
Impacted Files Coverage Ξ”
src/check/ast/mod.rs 93.69% <ΓΈ> (-0.06%) :arrow_down:
src/check/context/clss/generic.rs 93.18% <ΓΈ> (ΓΈ)
src/generate/ast/mod.rs 92.82% <ΓΈ> (-0.04%) :arrow_down:
src/generate/ast/node.rs 77.69% <ΓΈ> (+0.59%) :arrow_up:
src/parse/ast/mod.rs 100.00% <ΓΈ> (ΓΈ)
src/parse/ast/node.rs 79.77% <0.00%> (-0.11%) :arrow_down:
src/parse/block.rs 100.00% <ΓΈ> (ΓΈ)
src/parse/mod.rs 75.00% <83.33%> (+1.31%) :arrow_up:
src/parse/expr_or_stmt.rs 94.38% <87.50%> (-0.62%) :arrow_down:
src/check/ast/node.rs 95.75% <100.00%> (-0.02%) :arrow_down:
... and 5 more