Skip to content

Better handle coverage reporting of MacroIf nodes with _only_ MacroLiteral bodies#15938

Merged
Blacksmoke16 merged 1 commit intocrystal-lang:masterfrom
Blacksmoke16:macroif-macroliteral-coverage
Jun 30, 2025
Merged

Better handle coverage reporting of MacroIf nodes with _only_ MacroLiteral bodies#15938
Blacksmoke16 merged 1 commit intocrystal-lang:masterfrom
Blacksmoke16:macroif-macroliteral-coverage

Conversation

@Blacksmoke16
Copy link
Member

Resolves #15932

@Blacksmoke16 Blacksmoke16 added kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:tools:macro-code-coverage labels Jun 28, 2025
@straight-shoota straight-shoota added this to the 1.17.0 milestone Jun 30, 2025
@Blacksmoke16 Blacksmoke16 merged commit 8096f04 into crystal-lang:master Jun 30, 2025
40 checks passed
@Blacksmoke16 Blacksmoke16 deleted the macroif-macroliteral-coverage branch June 30, 2025 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:tools:macro-code-coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect macro code coverage for MacroIf with non-macro then/else blocks

2 participants