Thanks to visit codestin.com
Credit goes to github.com

Skip to content

tests: add subdir dvcignore test#11046

Open
legendof-selda wants to merge 5 commits into
treeverse:mainfrom
legendof-selda:fix/dvcignore_on_subdirs
Open

tests: add subdir dvcignore test#11046
legendof-selda wants to merge 5 commits into
treeverse:mainfrom
legendof-selda:fix/dvcignore_on_subdirs

Conversation

@legendof-selda
Copy link
Copy Markdown

@legendof-selda legendof-selda commented May 29, 2026

Thank you for the contribution - we'll try to review it as soon as possible. 🙏

This fixes Issue #11044

Now, having .dvcignore in the root of the SCM dir will be respected on the dependency files. This fix is especially useful for monorepos.

@github-project-automation github-project-automation Bot moved this to Backlog in DVC May 29, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 29, 2026

Codecov Report

❌ Patch coverage is 87.80488% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.97%. Comparing base (2431ec6) to head (b6ecdb0).
⚠️ Report is 204 commits behind head on main.

Files with missing lines Patch % Lines
dvc/dependency/base.py 84.61% 1 Missing and 1 partial ⚠️
dvc/repo/__init__.py 60.00% 1 Missing and 1 partial ⚠️
dvc/output.py 85.71% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11046      +/-   ##
==========================================
+ Coverage   90.68%   90.97%   +0.29%     
==========================================
  Files         504      505       +1     
  Lines       39795    41173    +1378     
  Branches     3141     3267     +126     
==========================================
+ Hits        36087    37459    +1372     
- Misses       3042     3073      +31     
+ Partials      666      641      -25     

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

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.

1 participant