Skip to content

Comments

Expose Missing Commons VFS Configuration Parameters#1501

Merged
niveathika merged 6 commits intoballerina-platform:masterfrom
SachinAkash01:add-vfs-configurations
Dec 3, 2025
Merged

Expose Missing Commons VFS Configuration Parameters#1501
niveathika merged 6 commits intoballerina-platform:masterfrom
SachinAkash01:add-vfs-configurations

Conversation

@SachinAkash01
Copy link
Member

@SachinAkash01 SachinAkash01 commented Oct 23, 2025

Purpose

$subject

Fixes: ballerina-platform/ballerina-library#8369

Proposal PR: ballerina-platform/ballerina-spec#1391

Examples

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests
  • Updated the spec
  • Checked native-image compatibility

@codecov
Copy link

codecov bot commented Oct 23, 2025

Codecov Report

❌ Patch coverage is 79.01235% with 34 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.66%. Comparing base (87620e1) to head (aec7116).
⚠️ Report is 7 commits behind head on master.

Files with missing lines Patch % Lines
...ain/java/io/ballerina/stdlib/ftp/util/FtpUtil.java 75.00% 5 Missing and 13 partials ⚠️
.../ftp/transport/server/util/FileTransportUtils.java 81.33% 5 Missing and 9 partials ⚠️
...java/io/ballerina/stdlib/ftp/client/FtpClient.java 77.77% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1501      +/-   ##
============================================
+ Coverage     80.53%   80.66%   +0.12%     
- Complexity      510      721     +211     
============================================
  Files            43       58      +15     
  Lines          2574     3274     +700     
  Branches        398      548     +150     
============================================
+ Hits           2073     2641     +568     
- Misses          345      397      +52     
- Partials        156      236      +80     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@niveathika niveathika added the Skip GraalVM Check This will skip the GraalVM compatibility check label Dec 1, 2025
@niveathika niveathika force-pushed the add-vfs-configurations branch 6 times, most recently from b6be18e to 0e3926d Compare December 2, 2025 18:16
@niveathika niveathika force-pushed the add-vfs-configurations branch from 0e3926d to d7f9613 Compare December 2, 2025 19:08
@niveathika niveathika force-pushed the add-vfs-configurations branch from fd73104 to a35db89 Compare December 3, 2025 17:33
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 3, 2025

@niveathika niveathika merged commit d3b14c2 into ballerina-platform:master Dec 3, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip GraalVM Check This will skip the GraalVM compatibility check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expose Missing Commons VFS Configuration Parameters in FTP Module

2 participants