c12 icon indicating copy to clipboard operation
c12 copied to clipboard

feat: `configFile: false` to disable config file loading

Open kricsleo opened this issue 7 months ago • 1 comments
trafficstars

resolves #236

This PR brings back support for configFile: false.

kricsleo avatar Apr 09 '25 14:04 kricsleo

Codecov Report

Attention: Patch coverage is 61.70213% with 18 lines in your changes missing coverage. Please review.

Project coverage is 77.79%. Comparing base (42d9697) to head (689ce39). Report is 131 commits behind head on main.

Files with missing lines Patch % Lines
src/watch.ts 0.00% 11 Missing :warning:
src/loader.ts 80.55% 7 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #245      +/-   ##
==========================================
+ Coverage   76.92%   77.79%   +0.87%     
==========================================
  Files           7        6       -1     
  Lines         806      635     -171     
  Branches       80      124      +44     
==========================================
- Hits          620      494     -126     
+ Misses        184      141      -43     
+ Partials        2        0       -2     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Apr 10 '25 07:04 codecov[bot]

Thanks for the time on this PR.

PR looks nice, but I'm a little bit hesitant if it adds real value vs internal complexity.

pi0 avatar Jul 30 '25 08:07 pi0

if it adds real value vs internal complexity.

Reasonable concern. We can just hold off on it until more users/real cases require this. ❤️

kricsleo avatar Jul 30 '25 10:07 kricsleo