[
https://issues.apache.org/jira/browse/GROOVY-11643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17948335#comment-17948335
] ASF GitHub Bot commented on GROOVY-11643: ----------------------------------------- codecov-commenter commented on PR #2208: URL: https://github.com/apache/groovy/pull/2208#issuecomment-2840642508 ## [Codecov](https://app.codecov.io/gh/apache/groovy/pull/2208?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report Attention: Patch coverage is `63.26531%` with `18 lines` in your changes missing coverage. Please review. > Project coverage is 68.9327%. Comparing base [(`1eeb735`)](https://app.codecov.io/gh/apache/groovy/commit/1eeb735e303fdb230feb03ea3b3f44f7e7750e66?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`3e2e390`)](https://app.codecov.io/gh/apache/groovy/commit/3e2e39083bffe250868ffbaa9558980b9cfd212f?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). > Report is 1 commits behind head on master. | [Files with missing lines](https://app.codecov.io/gh/apache/groovy/pull/2208?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines | |---|---|---| | [...rg/codehaus/groovy/runtime/ArrayGroovyMethods.java](https://app.codecov.io/gh/apache/groovy/pull/2208?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Fruntime%2FArrayGroovyMethods.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3J1bnRpbWUvQXJyYXlHcm9vdnlNZXRob2RzLmphdmE=) | 63.2653% | [12 Missing and 6 partials :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2208?src=pr&el=tree&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/2208?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) ```diff @@ Coverage Diff @@ ## master #2208 +/- ## ================================================== - Coverage 68.9387% 68.9327% -0.0059% - Complexity 29575 29583 +8 ================================================== Files 1423 1423 Lines 113952 114001 +49 Branches 19772 19781 +9 ================================================== + Hits 78557 78584 +27 - Misses 28787 28801 +14 - Partials 6608 6616 +8 ``` | [Files with missing lines](https://app.codecov.io/gh/apache/groovy/pull/2208?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [...rg/codehaus/groovy/runtime/ArrayGroovyMethods.java](https://app.codecov.io/gh/apache/groovy/pull/2208?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fcodehaus%2Fgroovy%2Fruntime%2FArrayGroovyMethods.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvY29kZWhhdXMvZ3Jvb3Z5L3J1bnRpbWUvQXJyYXlHcm9vdnlNZXRob2RzLmphdmE=) | `88.6884% <63.2653%> (-0.8949%)` | :arrow_down: | ... and [3 files with indirect coverage changes](https://app.codecov.io/gh/apache/groovy/pull/2208/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> > Add column, eachColumn to AGM > ----------------------------- > > Key: GROOVY-11643 > URL: https://issues.apache.org/jira/browse/GROOVY-11643 > Project: Groovy > Issue Type: Improvement > Reporter: Paul King > Assignee: Paul King > Priority: Major > Fix For: 5.0.0-alpha-13 > > > For original discussions, see: > https://lists.apache.org/thread/xv1bpgc7xp3rwp7qt627xfyd10ljcwbc > This issue splits out two new pieces of functionality for ArrayGroovyMethods > that emerged while working on the other issue. > The code: > {code:groovy} > matrix.column(1) > {code} > is a much more efficient, and arguably more intuitive, version of: > {code:groovy} > matrix*.getAt(1) > {code} > The code: > {code:groovy} > matrix.eachColumn { > // process column > } > {code} > is a much more efficient and slightly clearer version of: > {code:groovy} > matrix.transpose().each { > // process column > } > {code} -- This message was sent by Atlassian Jira (v8.20.10#820010)
