[ 
https://issues.apache.org/jira/browse/TINKERPOP-3143?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17947412#comment-17947412
 ] 

ASF GitHub Bot commented on TINKERPOP-3143:
-------------------------------------------

codecov-commenter commented on PR #3101:
URL: https://github.com/apache/tinkerpop/pull/3101#issuecomment-2831324618

   ## 
[Codecov](https://app.codecov.io/gh/apache/tinkerpop/pull/3101?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 Report
   All modified and coverable lines are covered by tests :white_check_mark:
   > Please [upload](https://docs.codecov.com/docs/codecov-uploader) report for 
BASE (`3.8-dev@45bf1ba`). [Learn 
more](https://docs.codecov.io/docs/error-reference?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#section-missing-base-commit)
 about missing BASE report.
   
   <details><summary>Additional details and impacted files</summary>
   
   
   ```diff
   @@            Coverage Diff             @@
   ##             3.8-dev    #3101   +/-   ##
   ==========================================
     Coverage           ?   75.03%           
   ==========================================
     Files              ?       28           
     Lines              ?     6157           
     Branches           ?        0           
   ==========================================
     Hits               ?     4620           
     Misses             ?     1329           
     Partials           ?      208           
   ```
   
   </details>
   
   [:umbrella: View full report in Codecov by 
Sentry](https://app.codecov.io/gh/apache/tinkerpop/pull/3101?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   
   :loudspeaker: Have feedback on the report? [Share it 
here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   
   <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>




> gremlin-javascript should default to GraphBinary
> ------------------------------------------------
>
>                 Key: TINKERPOP-3143
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-3143
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: javascript
>    Affects Versions: 3.7.3
>            Reporter: Stephen Mallette
>            Priority: Critical
>
> no idea how this wasn't done, but gremlin-javascript is still on GraphSONv3 
> as a default - should be GraphBinary like all the others. not breaking, but 
> should probably be done in a major rather than 3.7.x at this point. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to