Skip to content

docs: announce repository move - #5543

Merged
polarweasel merged 1 commit into
NVIDIA:mainfrom
polarweasel:ab/repository-move-notice
Aug 29, 2026
Merged

docs: announce repository move#5543
polarweasel merged 1 commit into
NVIDIA:mainfrom
polarweasel:ab/repository-move-notice

Conversation

@polarweasel

Copy link
Copy Markdown
Contributor

Adds pre-move notices to the README and documentation landing page for the September 4, 2026 transfer of NICo from NVIDIA/infra-controller to dsx-ai-factory/infra-controller. Infrastructure and canonical-link updates are intentionally deferred to cutover day.

(This is a redo of @deepak-poornachandra's #5466 that had some commit signing issues.)

Related Issues

N/A

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

N/A

@polarweasel
polarweasel requested a review from a team as a code owner August 28, 2026 23:17
@polarweasel polarweasel mentioned this pull request Aug 28, 2026
3 tasks
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 97005f01-b7e9-4c05-9d15-080a535e0db5

📥 Commits

Reviewing files that changed from the base of the PR and between 77355c5 and 962b7c0.

📒 Files selected for processing (4)
  • README.md
  • docs/index.md
  • fern/docs.yml
  • fern/fern.config.json
🚧 Files skipped from review as they are similar to previous changes (4)
  • fern/fern.config.json
  • README.md
  • fern/docs.yml
  • docs/index.md

Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review.


Summary by CodeRabbit

  • Documentation
    • Added notices about the repository moving to a new location on September 4, 2026.
    • Documented expected URL and Git operation redirects, plus guidance for updating automation and integration references.
    • Added the announcement to the documentation site and enabled canonical host metadata for improved discoverability.
  • Chores
    • Updated the documentation tooling configuration to the latest version.

Walkthrough

The change documents the planned repository move to dsx-ai-factory/infra-controller on September 4, 2026. It also updates the documentation site configuration and Fern version.

Changes

Repository relocation

Layer / File(s) Summary
Repository relocation notices
README.md, docs/index.md, fern/docs.yml
Adds relocation notices with the move date, redirect behavior, Git compatibility, and required integration updates.
Documentation configuration
fern/docs.yml, fern/fern.config.json
Enables canonical host metadata and updates Fern from 5.94.0 to 5.108.0.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 962b7

This PR adds repository-move notices and related documentation metadata without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description accurately summarizes the repository move notices, the September 4, 2026 cutover, deferred infrastructure updates, and documentation-only scope.
Title check ✅ Passed The title clearly and concisely identifies the primary change: announcing the repository move in documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@polarweasel
polarweasel requested a review from jayzhudev August 28, 2026 23:18
@github-actions

Copy link
Copy Markdown

Signed-off-by: Alex Ball <aball@nvidia.com>
@polarweasel
polarweasel force-pushed the ab/repository-move-notice branch from 4fc1227 to 962b7c0 Compare August 28, 2026 23:50
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.

@polarweasel
polarweasel enabled auto-merge (squash) August 28, 2026 23:54
Comment thread fern/docs.yml
message: "🔔 Infra Controller's GitHub repository is moving on September 4, 2026. Refer to the [front page](https://docs.nvidia.com/infra-controller) for more information."

metadata:
canonical-host: "docs.nvidia.com/infra-controller"

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.

Previously this was commented out. Is that important?

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.

It's an SEO thing. That host should be the default anyway, but I figured belt-and-suspenders can't hurt.

@polarweasel
polarweasel merged commit ecf13f2 into NVIDIA:main Aug 29, 2026
69 checks passed
@polarweasel
polarweasel deleted the ab/repository-move-notice branch August 29, 2026 00:24
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