Skip to content

Exclude example and test paths from Dependabot pip updates (#2856)#2859

Open
giac41 wants to merge 1 commit intopallets-eco:masterfrom
giac41:fix-dependabot-exclude-paths
Open

Exclude example and test paths from Dependabot pip updates (#2856)#2859
giac41 wants to merge 1 commit intopallets-eco:masterfrom
giac41:fix-dependabot-exclude-paths

Conversation

@giac41
Copy link
Copy Markdown

@giac41 giac41 commented Apr 14, 2026

Dependabot was scanning .txt files in example and test directories which are not actual dependency manifests.

The following paths are excluded from the pip ecosystem scan:

  • examples/**
  • flask_admin/tests/**

fixes #2856

…co#2856)

Dependabot was scanning .txt files in example and test directories which are not actual dependency manifests. This excludes those paths from the pip ecosystem scan.
@ElLorans
Copy link
Copy Markdown
Contributor

Why excluding flask_admin/tests/**? Shouldn't we exclude license, too?

@giac41
Copy link
Copy Markdown
Author

giac41 commented Apr 15, 2026

I excluded flask_admin/tests/** because I found several .txt files in test fixtures that clearly do not look like dependency manifests. However, I agree that excluding the whole directory is broader than necessary.

I checked recent Dependabot commits and did not find any that were clearly tied to .txt files, so I cannot confirm that these specific files have caused issues in practice.

That said, I found several .txt files in the repository that clearly do not appear to be dependency manifests:

  • ./LICENSE.txt
  • ./examples/bootstrap4/files/d1/dummy.txt
  • ./examples/bootstrap4/static/d1/afile.txt
  • ./examples/s3/localdir/yy/zz/afile.txt
  • ./flask_admin/tests/tmp/inner/test1.txt
  • ./flask_admin/tests/fileadmin/files/dummy.txt
  • ./flask_admin/tests/sqla/files/dummy.txt

So rather than excluding whole directories, I think an explicit exclude-paths list for these files would be a safer and more targeted approach.

@ElLorans
Copy link
Copy Markdown
Contributor

A simpler

 exclude-paths:
      - "*.txt"

would not require manually updating the list, though.

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.

Fix dependabot

2 participants