Skip to content

Add Dependabot configuration for pip and GitHub Actions#2041

Merged
rchiodo merged 2 commits into
microsoft:mainfrom
rchiodo:rchiodo-add-dependabot-config
Jun 24, 2026
Merged

Add Dependabot configuration for pip and GitHub Actions#2041
rchiodo merged 2 commits into
microsoft:mainfrom
rchiodo:rchiodo-add-dependabot-config

Conversation

@rchiodo

@rchiodo rchiodo commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a .github/dependabot.yml to enable automated dependency and security (CVE) update pull requests.

This addresses the recurring Component Governance / CVE alerts for the repository by letting Dependabot keep dependencies current and raise security update PRs automatically.

Ecosystems covered

Ecosystem Path Covers
pip /tests tests/requirements.txt (django, flask, gevent, numpy, requests, etc.)
github-actions / Actions used by the workflows (e.g. actions/checkout, github/codeql-action, actions/stale, actions/github-script)
  • Weekly schedule, grouped PRs to reduce noise, labels, and a 10 open-PR limit per ecosystem.

Notes

  • This file enables version updates. For Dependabot to also open PRs specifically for existing CVE alerts, Dependabot security updates must be enabled in repo Settings -> Code security.
  • Vendored dependencies under src/debugpy/_vendored/ and the git components in cgmanifest.json (pydevd, bytecode, winappdbg, scandir) are not managed by Dependabot and still require manual updates.

Enables automated dependency and security (CVE) update PRs for the pip
test requirements and the GitHub Actions used in workflows.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@rchiodo rchiodo requested a review from a team as a code owner June 24, 2026 17:19
Comment thread .github/dependabot.yml Outdated
# Keeps dependencies up to date and automatically opens pull requests for
# security (CVE) and version updates. Security updates are always enabled for
# every ecosystem Dependabot can detect; the entries below also enable regular
# version updates so dependencies don't drift far enough to accumulate

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The header comment states "Security updates are always enabled for every ecosystem Dependabot can detect," but this file only configures version updates. Dependabot security (CVE) updates require the separate toggle in repo Settings → Code security, as the PR description correctly notes. Consider softening this committed comment to match that caveat so a future maintainer doesn't assume CVE PRs are already flowing.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch - softened the header comment in 8d24d2c to make clear this file only configures version updates, and that CVE security updates require the separate Settings -> Code security toggle.

Comment thread .github/dependabot.yml Outdated
open-pull-requests-limit: 10
labels:
- "dependencies"
- "python"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The custom labels python and github-actions will be silently ignored by Dependabot if they don't already exist in the repo (only dependencies is auto-created). Either pre-create these labels or drop them to avoid a no-op.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Dropped the custom 'python' and 'github-actions' labels in 8d24d2c, keeping only the auto-created 'dependencies' label to avoid the silent no-op.

@heejaechang

Copy link
Copy Markdown
Contributor

Config looks sound and safe to merge. A couple of minor, non-blocking notes inline (misleading header comment about security updates, and custom labels that may need to be pre-created).

@heejaechang heejaechang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved via Review Center.

- Clarify header comment: this file enables version updates only; CVE
  security updates require the separate repo Settings toggle.
- Drop custom 'python' and 'github-actions' labels that Dependabot would
  silently ignore; keep only the auto-created 'dependencies' label.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Comment thread .github/dependabot.yml
open-pull-requests-limit: 10
labels:
- "dependencies"
groups:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The patterns: ["*"] grouping collapses every dependency into one all-or-nothing grouped PR. Given the PR's stated CVE-remediation goal, a single breaking major bump (e.g. numpy, gevent, django) can wedge the entire group and delay a security-relevant bump (e.g. requests) swept into it. Consider carving high-risk majors out of the catch-all group, or rely on Dependabot's ungrouped security PRs once that setting is enabled. Not a blocker.

@heejaechang

Copy link
Copy Markdown
Contributor

Config looks sound and safe to merge. One optional note about the catch-all grouping is worth considering, but it's non-blocking.

@heejaechang heejaechang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved via Review Center.

@rchiodo rchiodo merged commit e61c04e into microsoft:main Jun 24, 2026
7 of 27 checks passed
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