Add streaming put methods#1507
Conversation
Codecov Report❌ Patch coverage is ❌ Your project status has failed because the head coverage (68.42%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## master #1507 +/- ##
============================================
- Coverage 68.54% 68.42% -0.12%
- Complexity 292 445 +153
============================================
Files 34 49 +15
Lines 1860 2667 +807
Branches 245 414 +169
============================================
+ Hits 1275 1825 +550
- Misses 495 685 +190
- Partials 90 157 +67 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 15 out of 16 changed files in this pull request and generated 4 comments.
Comments suppressed due to low confidence (1)
native/src/main/java/io/ballerina/stdlib/ftp/client/FtpClient.java:1
- The previous code had a ternary operator checking
content is jsonwhich is now removed. While the new implementation is simpler, ensure that bothjsonandrecord {}types have atoJsonString()method to avoid potential runtime errors.
/*
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
native/src/main/java/io/ballerina/stdlib/ftp/ContentCsvStreamIteratorUtils.java
Show resolved
Hide resolved
native/src/main/java/io/ballerina/stdlib/ftp/ContentCsvStreamIteratorUtils.java
Outdated
Show resolved
Hide resolved
|



Purpose
Examples
Checklist