Skip to content

Do not exclude ".gitignore" and ".cvsignore" by default#326

Open
kwin wants to merge 1 commit intomasterfrom
feature/remove-gitignore-cvsignore-from-defaultexcludes
Open

Do not exclude ".gitignore" and ".cvsignore" by default#326
kwin wants to merge 1 commit intomasterfrom
feature/remove-gitignore-cvsignore-from-defaultexcludes

Conversation

@kwin
Copy link
Copy Markdown
Contributor

@kwin kwin commented Mar 30, 2026

This closes #312

@kwin kwin requested review from gnodet and slawekjaranowski March 30, 2026 16:41
Copy link
Copy Markdown
Member

@slawekjaranowski slawekjaranowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is ok
but we should mark change as breaking

@slawekjaranowski
Copy link
Copy Markdown
Member

also do it the same on 3.x branch

@kwin kwin added the breaking label Mar 30, 2026
@kwin
Copy link
Copy Markdown
Contributor Author

kwin commented Mar 31, 2026

also do it the same on 3.x branch

3.x does not contain .gitignore in excludes anyhow in that branch, only .csvignore needs to be done there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Revisit AbstractScanner.DEFAULT_EXCLUDES

2 participants