Skip to content

docs: remove moved Sweeper integration guide - #13470

Closed
jasonqinzhou wants to merge 2 commits into
ai-dynamo:mainfrom
jasonqinzhou:agent/remove-moved-sweeper-integration-doc
Closed

docs: remove moved Sweeper integration guide#13470
jasonqinzhou wants to merge 2 commits into
ai-dynamo:mainfrom
jasonqinzhou:agent/remove-moved-sweeper-integration-doc

Conversation

@jasonqinzhou

@jasonqinzhou jasonqinzhou commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the Dynamo Fern copy of the Sweeper integration guide after it moves to AISimulate
  • remove the page from Fern navigation and the Dynamo-owned-document integrity set
  • redirect the former dev-docs URL to the canonical AISimulate guide
  • update Dynamo's compatibility copy of the Sweeper examples to use the canonical cross-repository URL

Why

The guide is part of AISimulate's Sweeper documentation and is now maintained with the standalone package in ai-dynamo/aisimulate#6. Keeping a second manually maintained copy in Dynamo would recreate the ownership and link drift exposed by AISimulate PR #3.

Dependency

Merge after ai-dynamo/aisimulate#6 so the external redirect target exists on ai-dynamo/aisimulate@main.

Impact

Documentation only. The old Dynamo dev-docs URL remains usable through a permanent redirect.

Validation

  • targeted pre-commit suite passed, including YAML, docs sync integrity, asset paths, style, codespell, Ruff, and whitespace checks
  • exact GitHub Actions Markdown-link checker: 0 broken links across 580 files; 4 pre-existing suspicious-symlink warnings remain non-failing
  • python3 docs/fern/scripts/sync_aisimulate_docs.py --check
  • Ruby/Psych parse of docs/fern/index.yml and docs/fern/docs.yml
  • git diff --check

Signed-off-by: Jason Zhou (Engrg-Hardware 1) <jasonzho@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Aug 18, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@jasonqinzhou
jasonqinzhou temporarily deployed to external_collaborator August 18, 2026 16:54 — with GitHub Actions Inactive
@jasonqinzhou
jasonqinzhou temporarily deployed to external_collaborator August 18, 2026 16:54 — with GitHub Actions Inactive
@github-actions github-actions Bot added docs documentation Improvements or additions to documentation labels Aug 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

👋 Hi jasonqinzhou! Thank you for contributing to ai-dynamo/dynamo.

Just a reminder: The NVIDIA Test Github Validation CI runs an essential subset of the testing framework to quickly catch errors.Your PR reviewers may elect to test the changes comprehensively before approving your changes.

🚀

@github-actions github-actions Bot added the external-contribution Pull request is from an external contributor label Aug 18, 2026
Signed-off-by: Jason Zhou (Engrg-Hardware 1) <jasonzho@nvidia.com>
@jasonqinzhou
jasonqinzhou temporarily deployed to external_collaborator August 18, 2026 16:57 — with GitHub Actions Inactive
@dynamo-ops

Copy link
Copy Markdown
Contributor

Unsigned Commits Detected

The following commits are not GPG-signed and must be signed before CI can run:

Commit Reason
d136f0a unsigned
b42a316 unsigned

Please sign your commits and push again. See the GitHub docs on commit signature verification for help.

Copy link
Copy Markdown
Contributor Author

Superseded by #13473. The replacement uses a branch directly in ai-dynamo/dynamo so repository maintainers can update the PR collaboratively, and it is rebased onto the current upstream main.

@jasonqinzhou
jasonqinzhou deleted the agent/remove-moved-sweeper-integration-doc branch August 18, 2026 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs documentation Improvements or additions to documentation external-contribution Pull request is from an external contributor size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants