Skip to content

ci: stop scanning test/ directory with dependabot#5651

Open
nancysangani wants to merge 1 commit intoossf:mainfrom
nancysangani:ci/stop-scanning-test-dir-dependabot
Open

ci: stop scanning test/ directory with dependabot#5651
nancysangani wants to merge 1 commit intoossf:mainfrom
nancysangani:ci/stop-scanning-test-dir-dependabot

Conversation

@nancysangani
Copy link
Copy Markdown

Closes #5253

The pip ecosystem entry with directory: "/" would scan any requirements files found under test/, generating false positive
alerts for packages that are only used as test fixtures for the vulnerability scanner itself — they are never installed or run.

Add an ignore block with a paths filter to exclude everything under test/** from dependabot's pip scanning, while keeping dependabot active for the actual project dependencies.

Signed-off-by: Nancy <9d.24.nancy.sangani@gmail.com>
Copilot AI review requested due to automatic review settings March 20, 2026 16:22
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Dependabot configuration to prevent pip dependency scanning from flagging test-fixture requirement files under test/, reducing false-positive vulnerability alerts while keeping Dependabot enabled for real project dependencies.

Changes:

  • Removes previously commented-out ecosystem entries intended to suppress /test/ scanning.
  • Adds an ignore rule to the pip update configuration intended to exclude test/**.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +15 to +18
ignore:
- dependency-name: "*"
paths:
- "test/**"
Copy link

Copilot AI Mar 20, 2026

Choose a reason for hiding this comment

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

The new ignore rule uses dependency-name: "*" with a paths: filter. Dependabot's ignore entries are evaluated by dependency name/version/update-type; if paths is not a supported key (and gets ignored), this will effectively ignore all pip dependency updates for the repo, disabling the intended weekly pip updates. Please verify Dependabot supports paths here; if not, a safer approach is to scope the pip update directory to only the folder(s) containing real project manifests, or rename/move the test fixture requirement files so they are not detected as pip manifests.

Suggested change
ignore:
- dependency-name: "*"
paths:
- "test/**"

Copilot uses AI. Check for mistakes.
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.

ci: stop scanning test/ directory with dependabot

2 participants