Skip to content

docs: fix torchforge link in the MCP environment tutorial - #1255

Closed
pratikgx wants to merge 2 commits into
openenv:mainfrom
pratikgx:docs-fix-torchforge-link
Closed

pratikgx wants to merge 2 commits into
openenv:mainfrom
pratikgx:docs-fix-torchforge-link

Conversation

@pratikgx

@pratikgx pratikgx commented Sep 27, 2026 •

Copy link
Copy Markdown

Summary

The MCP environment tutorial links github.com/pytorch-labs/torchforge, which now 404s; the repository lives at github.com/meta-pytorch/torchforge (the README's torchforge link already points at the meta-pytorch site).

Type of Change

  • Documentation

Alignment Checklist

  • Read PRINCIPLES.md; a link repair has no design impact.
  • Checked INVARIANTS.md; none touched.
  • Lint/tests: not applicable to a one-line Markdown change.

RFC Status

  • Not required (docs)

Test Plan

Old URL returns 404, new URL returns 200 (checked with curl following redirects).

Claude Code Review

Claude Code assisted with the link check and edit; /alignment-review was not run for a one-line docs fix.


Note

Low Risk
Documentation-only URL change with no runtime or API impact.

Overview
Updates the torchforge GitHub link in the MCP environment tutorial (mcp-environment.md) from the dead github.com/pytorch-labs/torchforge URL to github.com/meta-pytorch/torchforge, matching where the project lives now and aligning with other docs (e.g. the training guide).

Reviewed by Cursor Bugbot for commit cc103c6. Bugbot is set up for automated code reviews on this repo. Configure here.

The repository moved from pytorch-labs/torchforge to
meta-pytorch/torchforge; the old URL returns 404.

@cursor cursor Bot 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.

Reviewed at 31026897. This is a single link fix, and https://github.com/meta-pytorch/torchforge resolves (HTTP 200). The branch merges cleanly into current main.

This can't merge yet because the repository workflows for this fork (docs build, tests, package CI) are still waiting on a maintainer's Approve and run. I'll merge it once exact-head CI is green.

Open in Web View Automation 

Sent by Cursor Automation: Release

@cursor cursor Bot mentioned this pull request Sep 28, 2026
4 of 16 tasks
@burtenshaw

Copy link
Copy Markdown
Collaborator

Closing as superseded by #1297 (436ee3a), which landed this exact link correction. After refreshing against main, this PR has no remaining diff.

@burtenshaw burtenshaw closed this Oct 19, 2026
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