Skip to content

Conversation

@tony
Copy link
Member

@tony tony commented Nov 15, 2025

Changes

Bug Fixes

vcspull discover: Success logs redact config paths

Writes triggered by vcspull discover now pass the config file through
PrivatePath, so confirmations like "✓ Successfully updated ~/.vcspull.yaml"
and dry-run notices no longer leak the absolute home directory.

vcspull discover: Fix another workspace dir case

Discover now inspects the config scope (user/system/project) before writing,
so user-level configs like ~/.vcspull.yaml prefer tilde-prefixed workspace
keys while project-level configs keep their relative ./ sections. Tests
cover both behaviors to guard against regressions.

@codecov
Copy link

codecov bot commented Nov 15, 2025

Codecov Report

❌ Patch coverage is 97.50000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 78.52%. Comparing base (14e3002) to head (78fd3fe).
⚠️ Report is 10 commits behind head on master.

Files with missing lines Patch % Lines
src/vcspull/cli/discover.py 97.50% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #487      +/-   ##
==========================================
+ Coverage   77.95%   78.52%   +0.56%     
==========================================
  Files          14       14              
  Lines        1901     1937      +36     
  Branches      401      404       +3     
==========================================
+ Hits         1482     1521      +39     
+ Misses        272      271       -1     
+ Partials      147      145       -2     

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

@tony tony force-pushed the discover-pwd branch 2 times, most recently from 014ddd1 to 33f770a Compare November 15, 2025 11:51
@tony tony merged commit 4eee157 into master Nov 15, 2025
9 checks passed
@tony tony deleted the discover-pwd branch November 15, 2025 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants