Repository navigation
Point the SSH password scripts at runpod/pod-ssh-password - #952
Merged
Merged
Conversation
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
justinwlin
marked this pull request as ready for review
November 6, 2026 04:48
OneAsterisk
approved these changes
Nov 6, 2026
OneAsterisk
left a comment
There was a problem hiding this comment.
Claude review, round 1
Verdict: Approved — two pages now point at runpod/pod-ssh-password; every new URL resolves and the scripts behave the same.
Approval gate: 0 Critical, 0 High, 0 Medium+Low → approved.
Existing feedback
- mintlify[bot]: preview deployment only, no claims. The
Mintlify Deploymentstatus reports skipped at this head while the preview comment shows the preview as Ready; nothing to act on.
PR description vs code
| Claim | Status | Where |
|---|---|---|
Links and wget commands updated on pods/configuration/use-ssh.mdx |
✅ | use-ssh.mdx:146, use-ssh.mdx:149 |
Links and wget commands updated on community-solutions/ssh-password-migration/overview.mdx |
✅ | overview.mdx:12, overview.mdx:49, overview.mdx:64 |
| Both raw URLs return 200 | ✅ | raw.githubusercontent.com/runpod/pod-ssh-password/main/passwordrunpod.sh and .../SCPMigration both 200; the blob/main/passwordrunpod.sh page link also 200 |
| "The scripts are the same. The only changes are their own URLs and placeholder IPs in the README." | Functionally identical. passwordrunpod.sh changes are URL-only (header lines 4-5, self-referencing wget at line 317). SCPMigration also changed "RunPod" to "Runpod" in the docstring, SCRIPT_NAME (line 17), and the welcome banner (line 551), and dropped the Author: Assistant line. No logic changes. |
Findings
None.
Nits
- PR body: "the scripts are the same" slightly understates the
SCPMigrationdiff (casing and author-line changes, see table above). Worth one clause so the next reader does not have to diff the repos.
Simplification opportunities
- Follow-up, not this PR: the new README switched its sample output to RFC 5737 placeholder IPs (
203.0.113.x), butoverview.mdx:54,overview.mdx:93, anduse-ssh.mdx:100,:164,:171,:174,:177still show real-looking public IPs. Same treatment there would keep the docs consistent with the repo. Pre-existing onmain, so out of scope here.
Verified clean
- The diff touches only the five URL lines; no unrelated changes in either page.
runpod/pod-ssh-passwordis public, unarchived, default branchmain, and containsREADME.md,SCPMigration,passwordrunpod.sh.- The
SCPMigrationURL dropped therefs/heads/mainform in favor ofmain; both resolve, andmainmatches the other links. - No remaining references to
justinwlin/Runpod-SSH-Password,passwordrunpod, orSCPMigrationanywhere else in the repo (docs.jsonandcommunity-solutions/overview.mdxonly reference the docs page path, which is unchanged). Check tooltip importspassed at this head.- The two other
justinwlin/*repos still linked from the docs (Runpod-Network-Volume-Storage-Tool,Runpod-GPU-And-Serverless-Base) are different projects and are not part of this move.
Questions for the author
justinwlin/Runpod-SSH-Passwordis still live and not archived, so it is now a second copy rather than a redirect. Is the plan to archive it or add a pointer to the org repo, so the two do not drift?
Reviewed by Claude (claude-fable-5-1) via the pr-review skill. Tokens: 228 in / 5,451 out / 493,982 cache read / 72,829 cache write = 572,490. Estimated cost: $1.86 at $10/$50 per MTok in/out, $0.25 cache read, $20 cache write (1-hour TTL, 2× input). Round 1 of this PR. This is automated assistance; a human owns the final call.
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The SSH password and pod-to-pod transfer scripts moved from a personal repo to runpod/pod-ssh-password. This updates the links and
wgetcommands on two pages:pods/configuration/use-ssh.mdxcommunity-solutions/ssh-password-migration/overview.mdxThe scripts are the same. The only changes are their own URLs and placeholder IPs in the README. Both raw URLs return 200.