Skip to content

Remove the local update_deps submodule propagation tool - #477

Merged
Dan Cristoloveanu (dcristoloveanu) merged 1 commit into
masterfrom
dev/dcristo/remove-update-deps
Oct 23, 2026
Merged

Dan Cristoloveanu (dcristoloveanu) merged 1 commit into
masterfrom
dev/dcristo/remove-update-deps

Conversation

@dcristoloveanu

Copy link
Copy Markdown
Member

Removes update_deps/, the locally-run submodule propagation tool. It is superseded by the dependency propagation pipeline in Azure-MessagingStore (build/propagate_deps.yml), which runs as the build service so the PRs are bot-authored.

Why this is safe

  • The pipeline runs Azure-MessagingStore's own copy (scripts/update_deps/propagate_updates.ps1), not deps/c-build-tools/update_deps.
  • That copy is self-contained: it builds the graph live via BFS and reads its own ignores.json. It does not fetch known_graph.json from this repo's raw GitHub URL (the old build_graph.ps1 here did).
  • An ADO code search across the One project for propagate_updates returns only Azure-MessagingStore's own copy and pipeline. Nothing else invokes this folder.
  • Nothing else in this repo (CMake, pipelines, validation, tests) references update_deps.

Changes

  • Delete update_deps/ (scripts, docs, ignores.json, known_graph.json, images).
  • Drop the "Dependency Management / Update Propagation" section from .github/copilot-instructions.md.

Related open PRs

These open PRs modify update_deps/. Merging this one makes them moot: #474, #473, #410.

The update_deps scripts are superseded by the dependency propagation pipeline in
Azure-MessagingStore (build/propagate_deps.yml), which runs its own copy under
scripts/update_deps as the build service so the PRs are bot-authored. That copy
is self-contained: it discovers the graph live and reads its own ignores.json, so
nothing depends on this folder, including the known_graph.json raw URL.

Delete update_deps and drop its section from the Copilot instructions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@dcristoloveanu

Copy link
Copy Markdown
Member Author

/azp run Azure-C-Build-Tools-Gate

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@dcristoloveanu
Dan Cristoloveanu (dcristoloveanu) merged commit 1e4ca34 into master Oct 23, 2026
12 checks passed
@dcristoloveanu
Dan Cristoloveanu (dcristoloveanu) deleted the dev/dcristo/remove-update-deps branch October 23, 2026 04:40
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