Skip to content

chore: enable dependabot cofig#17561

Open
dmcilvaney wants to merge 4 commits into
microsoft:4.0from
dmcilvaney:damcilva/4.0/pipelines/dependabot_config
Open

chore: enable dependabot cofig#17561
dmcilvaney wants to merge 4 commits into
microsoft:4.0from
dmcilvaney:damcilva/4.0/pipelines/dependabot_config

Conversation

@dmcilvaney
Copy link
Copy Markdown
Contributor

@dmcilvaney dmcilvaney commented May 29, 2026

dependabot doesn't like to make changes to sensitive folders (.github/workflows for example). Move the scripts out of that part of the tree into the general scripts folder.
Also add version pins to all current deps and add a basic smoke test to ensure updates don't obviously break things.

Copilot AI review requested due to automatic review settings May 29, 2026 23:04
Copy link
Copy Markdown
Contributor

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

Adds a Dependabot configuration to keep Python tooling dependencies (pip) and GitHub Actions used by workflows up to date on a weekly cadence.

Changes:

  • Configures pip ecosystem updates for requirements.txt files under .github/workflows/scripts/* and /scripts/* using globbing-capable directories.
  • Configures github-actions ecosystem updates at repo root, grouping all actions updates into a single PR via the actions group.
  • Sets weekly schedule, PR limit of 5, and applies dependencies plus ecosystem-specific labels.

@dmcilvaney dmcilvaney marked this pull request as draft May 29, 2026 23:45
@dmcilvaney dmcilvaney force-pushed the damcilva/4.0/pipelines/dependabot_config branch from 747a373 to c24f72d Compare May 30, 2026 00:37
Dependabot's pip updater cannot open PRs that modify any file under .github/
(only the github-actions ecosystem is granted that scope), so requirements.txt
under .github/workflows/scripts/ could never be auto-updated. Relocate the
helper-scripts subtree to top-level scripts/ci/ — linted (ruff/pyright scan
scripts/) and writable by Dependabot. Pure rename; reference updates follow.
@dmcilvaney dmcilvaney force-pushed the damcilva/4.0/pipelines/dependabot_config branch 2 times, most recently from 1216089 to 073ca27 Compare May 30, 2026 01:55
@dmcilvaney dmcilvaney marked this pull request as ready for review May 30, 2026 01:55
@dmcilvaney dmcilvaney requested a review from Copilot May 30, 2026 01:55
Copy link
Copy Markdown
Contributor

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

Copilot reviewed 15 out of 30 changed files in this pull request and generated no new comments.

Update all callers, dependabot.yml, and instruction docs to the new
scripts/ci/ location. dependabot pip ecosystem uses a single recursive
/scripts/** glob covering scripts/ci/{control-tower,spec-review} and
scripts/mcps. Add cooldown (delay new releases to catch bad ones) and
group all python tooling bumps into one PR.
Pin the previously-loose pip deps (pydantic==2.13.4, mcp==1.27.2,
python-dotenv==1.2.2) so Dependabot bumps from a concrete baseline.
Add a Dependency Smoke Test workflow that installs each requirements.txt
in a clean venv, py_compiles the scripts, and imports them to catch a
bad bump (or new python) breaking the helper scripts. Triggers on
requirements.txt and *.py changes under scripts/.
@dmcilvaney dmcilvaney force-pushed the damcilva/4.0/pipelines/dependabot_config branch from 073ca27 to d02a2ba Compare May 30, 2026 01:59
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