[
https://issues.apache.org/jira/browse/GROOVY-11976?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18077260#comment-18077260
] ASF GitHub Bot commented on GROOVY-11976: ----------------------------------------- codecov-commenter commented on PR #2503: URL: https://github.com/apache/groovy/pull/2503#issuecomment-4350575489 ## [Codecov](https://app.codecov.io/gh/apache/groovy/pull/2503?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report :x: Patch coverage is `73.59199%` with `211 lines` in your changes missing coverage. Please review. :white_check_mark: Project coverage is 67.1337%. Comparing base ([`839717c`](https://app.codecov.io/gh/apache/groovy/commit/839717c23639ba178e67c029a776ed99f42940d0?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`5834f22`](https://app.codecov.io/gh/apache/groovy/commit/5834f22577a072db5a25c77d4ed888ca9741c3e3?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)). | [Files with missing lines](https://app.codecov.io/gh/apache/groovy/pull/2503?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines | |---|---|---| | [.../codehaus/groovy/transform/DelegateASTStubber.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FDelegateASTStubber.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9EZWxlZ2F0ZUFTVFN0dWJiZXIuamF2YQ==) | 50.7692% | [50 Missing and 14 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...us/groovy/transform/IndexedPropertyASTStubber.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FIndexedPropertyASTStubber.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9JbmRleGVkUHJvcGVydHlBU1RTdHViYmVyLmphdmE=) | 60.5263% | [7 Missing and 8 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [.../groovy/groovy/beans/ListenerListASTStubber.groovy](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fgroovy%2Fgroovy%2Fbeans%2FListenerListASTStubber.groovy&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vZ3Jvb3Z5L2dyb292eS9iZWFucy9MaXN0ZW5lckxpc3RBU1RTdHViYmVyLmdyb292eQ==) | 67.5000% | [1 Missing and 12 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...va/groovy/transform/builder/BuilderASTStubber.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Fgroovy%2Ftransform%2Fbuilder%2FBuilderASTStubber.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9ncm9vdnkvdHJhbnNmb3JtL2J1aWxkZXIvQnVpbGRlckFTVFN0dWJiZXIuamF2YQ==) | 81.3559% | [1 Missing and 10 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [.../groovy/transform/RecordTypeASTTransformation.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FRecordTypeASTTransformation.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9SZWNvcmRUeXBlQVNUVHJhbnNmb3JtYXRpb24uamF2YQ==) | 56.0000% | [0 Missing and 11 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [src/main/java/groovy/beans/BindableASTStubber.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Fgroovy%2Fbeans%2FBindableASTStubber.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9ncm9vdnkvYmVhbnMvQmluZGFibGVBU1RTdHViYmVyLmphdmE=) | 75.0000% | [0 Missing and 9 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [src/main/java/groovy/beans/VetoableASTStubber.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Fgroovy%2Fbeans%2FVetoableASTStubber.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9ncm9vdnkvYmVhbnMvVmV0b2FibGVBU1RTdHViYmVyLmphdmE=) | 75.6757% | [0 Missing and 9 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...org/codehaus/groovy/transform/FinalASTStubber.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FFinalASTStubber.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9GaW5hbEFTVFN0dWJiZXIuamF2YQ==) | 43.7500% | [5 Missing and 4 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...ehaus/groovy/transform/NamedVariantASTStubber.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FNamedVariantASTStubber.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9OYW1lZFZhcmlhbnRBU1RTdHViYmVyLmphdmE=) | 63.6364% | [1 Missing and 7 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...odehaus/groovy/transform/RecordBaseASTStubber.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FRecordBaseASTStubber.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9SZWNvcmRCYXNlQVNUU3R1YmJlci5qYXZh) | 84.0909% | [0 Missing and 7 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | ... and [16 more](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | <details><summary>Additional details and impacted files</summary> [](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) ```diff @@ Coverage Diff @@ ## master #2503 +/- ## ================================================== + Coverage 67.0917% 67.1337% +0.0420% - Complexity 31765 31941 +176 ================================================== Files 1465 1486 +21 Lines 123373 124121 +748 Branches 22107 22318 +211 ================================================== + Hits 82773 83327 +554 - Misses 33493 33563 +70 - Partials 7107 7231 +124 ``` | [Files with missing lines](https://app.codecov.io/gh/apache/groovy/pull/2503?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [.../groovy/beans/ListenerListASTTransformation.groovy](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fgroovy%2Fgroovy%2Fbeans%2FListenerListASTTransformation.groovy&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vZ3Jvb3Z5L2dyb292eS9iZWFucy9MaXN0ZW5lckxpc3RBU1RUcmFuc2Zvcm1hdGlvbi5ncm9vdnk=) | `89.2562% <100.0000%> (+1.0209%)` | :arrow_up: | | [...n/java/groovy/beans/BindableASTTransformation.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Fgroovy%2Fbeans%2FBindableASTTransformation.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9ncm9vdnkvYmVhbnMvQmluZGFibGVBU1RUcmFuc2Zvcm1hdGlvbi5qYXZh) | `92.9134% <100.0000%> (+0.2904%)` | :arrow_up: | | [...n/java/groovy/beans/VetoableASTTransformation.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Fgroovy%2Fbeans%2FVetoableASTTransformation.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9ncm9vdnkvYmVhbnMvVmV0b2FibGVBU1RUcmFuc2Zvcm1hdGlvbi5qYXZh) | `92.7007% <100.0000%> (+0.2765%)` | :arrow_up: | | [...s/groovy/tools/javac/JavaAwareCompilationUnit.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftools%2Fjavac%2FJavaAwareCompilationUnit.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3Rvb2xzL2phdmFjL0phdmFBd2FyZUNvbXBpbGF0aW9uVW5pdC5qYXZh) | `89.4737% <100.0000%> (+0.7413%)` | :arrow_up: | | [...ansform/ASTTransformationCollectorCodeVisitor.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FASTTransformationCollectorCodeVisitor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9BU1RUcmFuc2Zvcm1hdGlvbkNvbGxlY3RvckNvZGVWaXNpdG9yLmphdmE=) | `91.1392% <100.0000%> (ø)` | | | [...aus/groovy/transform/ASTTransformationVisitor.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FASTTransformationVisitor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9BU1RUcmFuc2Zvcm1hdGlvblZpc2l0b3IuamF2YQ==) | `81.5217% <100.0000%> (+0.2031%)` | :arrow_up: | | [...s/groovy/transform/AutoCloneASTTransformation.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FAutoCloneASTTransformation.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9BdXRvQ2xvbmVBU1RUcmFuc2Zvcm1hdGlvbi5qYXZh) | `53.3742% <100.0000%> (+1.4755%)` | :arrow_up: | | [...oovy/transform/AutoImplementASTTransformation.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FAutoImplementASTTransformation.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9BdXRvSW1wbGVtZW50QVNUVHJhbnNmb3JtYXRpb24uamF2YQ==) | `88.0597% <100.0000%> (+0.4628%)` | :arrow_up: | | [...aus/groovy/transform/BuilderASTTransformation.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FBuilderASTTransformation.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9CdWlsZGVyQVNUVHJhbnNmb3JtYXRpb24uamF2YQ==) | `64.3836% <100.0000%> (+1.7936%)` | :arrow_up: | | [...us/groovy/transform/DelegateASTTransformation.java](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Ftransform%2FDelegateASTTransformation.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3RyYW5zZm9ybS9EZWxlZ2F0ZUFTVFRyYW5zZm9ybWF0aW9uLmphdmE=) | `87.7049% <100.0000%> (+0.2572%)` | :arrow_up: | | ... and [33 more](https://app.codecov.io/gh/apache/groovy/pull/2503?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | ... and [6 files with indirect coverage changes](https://app.codecov.io/gh/apache/groovy/pull/2503/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) </details> <details><summary> :rocket: New features to boost your workflow: </summary> - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems. - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges. </details> > stub generation should better account for the shape that will exist after > later transforms run when known > --------------------------------------------------------------------------------------------------------- > > Key: GROOVY-11976 > URL: https://issues.apache.org/jira/browse/GROOVY-11976 > Project: Groovy > Issue Type: Improvement > Reporter: Paul King > Assignee: Paul King > Priority: Major > > https://groovy.apache.org/wiki/GEP-21.html -- This message was sent by Atlassian Jira (v8.20.10#820010)
