search

JSAbrahams / mamba

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

Add panic messages and use unimplemented macro in desugar stage #81

Closed JSAbrahams closed 5 years ago

JSAbrahams commented 5 years ago

Relevant issues

...

Summary

Make more explicit what should and shouldn't be encountered in the desugar stage. This makes it easier to see on first glance what can and can't be desugared, and what should and shouldn't be encountered.

Added Tests

...

Additional Context

...

codecov[bot] commented 5 years ago

Codecov Report

Merging #81 into master will increase coverage by 0.26%. The diff coverage is 89.06%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   81.36%   81.63%   +0.26%     
==========================================
  Files          61       62       +1     
  Lines        3982     4040      +58     
==========================================
+ Hits         3240     3298      +58     
  Misses        742      742
Impacted Files Coverage Δ
src/parser/ast.rs 24.46% <ø> (+0.51%) :arrow_up:
src/desugar/node.rs 68.07% <62.5%> (-0.2%) :arrow_down:
tests/desugar/definition.rs 87.25% <90.9%> (+0.35%) :arrow_up:
tests/desugar/expression_and_statements.rs 90.66% <92.3%> (+0.34%) :arrow_up:
tests/desugar/types.rs 95.23% <95.23%> (ø)
src/core/mod.rs 69.48% <0%> (+1.94%) :arrow_up:

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 d4782a2...7c7ede8. Read the comment docs.