search

JSAbrahams / mamba

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

Move mutable field from VarDef to IdType #60

Closed JSAbrahams closed 5 years ago

JSAbrahams commented 5 years ago

Relevant issues

27

Summary

This reflects the change in the grammar. A variable definition does not contain any information about the mutability of the definition. This is now stored in the id alongside type information.

Added Tests

Modified existing tests.

Additional Context

...

codecov[bot] commented 5 years ago

Codecov Report

Merging #60 into master will decrease coverage by 0.04%. The diff coverage is 90%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
- Coverage   80.14%   80.09%   -0.05%     
==========================================
  Files          57       57              
  Lines        3888     3894       +6     
==========================================
+ Hits         3116     3119       +3     
- Misses        772      775       +3
Impacted Files Coverage Ξ”
tests/desugar/definition.rs 88% <ΓΈ> (ΓΈ) :arrow_up:
tests/parser/valid/call.rs 85.41% <100%> (ΓΈ) :arrow_up:
tests/parser/valid/definition.rs 89.5% <100%> (ΓΈ) :arrow_up:
src/desugar/node.rs 69.51% <100%> (ΓΈ) :arrow_up:
src/parser/ast.rs 23.94% <100%> (ΓΈ) :arrow_up:
src/parser/_type.rs 79.71% <100%> (+0.76%) :arrow_up:
tests/desugar/collection.rs 70.83% <100%> (ΓΈ) :arrow_up:
src/parser/definition.rs 75.92% <76.92%> (-1.72%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data Powered by Codecov. Last update ea4d805...cd5d6c2. Read the comment docs.