Closed seanyoung closed 10 months ago
Attention: 49 lines
in your changes are missing coverage. Please review.
Comparison is base (
8c4111b
) 87.74% compared to head (8190ca8
) 87.89%. Report is 1 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
solang-parser/src/helpers/fmt.rs | 20.83% | 38 Missing :warning: |
solang-parser/src/pt.rs | 0.00% | 6 Missing :warning: |
src/sema/ast.rs | 78.57% | 3 Missing :warning: |
src/sema/dotgraphviz.rs | 98.71% | 1 Missing :warning: |
src/sema/mod.rs | 98.93% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Merging for #1560
This simplifies the code, and makes it to use ExprContext for other purposes (for example variable scoping).