Skip to content

Simplify ast control stmts#875

Merged
riederm merged 11 commits intomasterfrom
simplyfy_ast_control_stmts
Jul 19, 2023
Merged

Simplify ast control stmts#875
riederm merged 11 commits intomasterfrom
simplyfy_ast_control_stmts

Conversation

@riederm
Copy link
Copy Markdown
Collaborator

@riederm riederm commented Jun 13, 2023

No description provided.

@riederm riederm changed the title Simplyfy ast control stmts Simplify ast control stmts Jun 13, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 13, 2023

Codecov Report

Patch coverage: 94.63% and project coverage change: -0.01 ⚠️

Comparison is base (9b89598) 96.12% compared to head (838b6b1) 96.12%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #875      +/-   ##
==========================================
- Coverage   96.12%   96.12%   -0.01%     
==========================================
  Files         116      117       +1     
  Lines       38392    38480      +88     
==========================================
+ Hits        36904    36987      +83     
- Misses       1488     1493       +5     
Impacted Files Coverage Δ
src/ast/control_statements.rs 50.00% <50.00%> (ø)
src/validation/statement.rs 92.57% <62.06%> (+0.47%) ⬆️
src/ast.rs 94.64% <100.00%> (+0.61%) ⬆️
src/ast/pre_processor.rs 99.03% <100.00%> (ø)
src/codegen/generators/statement_generator.rs 97.74% <100.00%> (+0.03%) ⬆️
src/parser/control_parser.rs 95.45% <100.00%> (ø)
src/resolver.rs 97.24% <100.00%> (-0.01%) ⬇️

... and 6 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@riederm riederm force-pushed the simplyfy_ast_control_stmts branch 3 times, most recently from 73078c1 to 0459763 Compare July 8, 2023 19:21
@riederm riederm force-pushed the simplyfy_ast_control_stmts branch from 0459763 to 871cc08 Compare July 8, 2023 19:44
Comment thread src/ast.rs Outdated
@ghaith ghaith mentioned this pull request Jul 13, 2023
@riederm riederm requested a review from ghaith July 18, 2023 16:47
@riederm riederm merged commit 38f90f6 into master Jul 19, 2023
@riederm riederm deleted the simplyfy_ast_control_stmts branch July 19, 2023 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants