Skip to content

Feat/no service config builder out dir#650

Open
junliurs wants to merge 1 commit into
mainfrom
feat/no-service-config-builder-out-dir
Open

Feat/no service config builder out dir#650
junliurs wants to merge 1 commit into
mainfrom
feat/no-service-config-builder-out-dir

Conversation

@junliurs
Copy link
Copy Markdown

@junliurs junliurs commented May 9, 2026

Motivation

Solution

@junliurs junliurs requested review from a team as code owners May 9, 2026 08:39
@codecov
Copy link
Copy Markdown

codecov Bot commented May 12, 2026

Codecov Report

❌ Patch coverage is 71.42857% with 46 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.11%. Comparing base (d7acebc) to head (d21f6d5).

Files with missing lines Patch % Lines
volo-build/src/workspace.rs 0.00% 30 Missing ⚠️
volo-build/src/config_builder.rs 86.25% 11 Missing ⚠️
volo-build/src/thrift_backend.rs 0.00% 4 Missing ⚠️
volo-build/src/util.rs 97.29% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #650      +/-   ##
==========================================
+ Coverage   44.87%   46.11%   +1.23%     
==========================================
  Files         162      162              
  Lines       20081    20216     +135     
==========================================
+ Hits         9011     9322     +311     
+ Misses      11070    10894     -176     

☔ 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.

@junliurs junliurs force-pushed the feat/no-service-config-builder-out-dir branch from a588842 to 9198793 Compare May 12, 2026 10:20
@junliurs junliurs force-pushed the feat/no-service-config-builder-out-dir branch from 9198793 to d21f6d5 Compare May 12, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant