This repository was archived by the owner on Aug 14, 2026. It is now read-only.
docs: pin the migration guide link to a commit instead of main - #529
Merged
Conversation
This repository is archived, so its README cannot be corrected later without unarchiving. A blob/main link breaks silently if developerDocs/stream-migration.md is ever moved or removed from the SDK repo, leaving the only pointer off this page dead with no way to fix it. Pinned to 5be5f2b, the commit that introduced the file in the v11.9.0 sync. The v11.9.0 tag points at the same commit; the SHA is used because a tag can be deleted and a reachable commit cannot. The npm deprecation message for @opensea/stream-js is being updated to the same pinned URL for the same reason. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
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.
Follow-up to #528. That PR's migration-guide link points at
blob/main, which breaks silently ifdeveloperDocs/stream-migration.mdis ever moved or removed from the SDK repo.That matters more here than in a normal repo: this one is archived, so the README cannot be corrected later without unarchiving again. The notice at the top would be left with a dead link and no way to fix it.
Pinned to
5be5f2bd3c5a4475d5a23eb8e0f5ca359f3df41e, the commit that introduced the file in the v11.9.0 sync. Verified it returns 200.The
v11.9.0tag points at the same commit and would read better, but a tag can be deleted and a reachable commit cannot, so the SHA wins for something that can't be edited afterward.Not changed
[license-link]also usesblob/main, and that one is correct: it points at this repo's ownLICENSE, and an archived repo'smainnever moves. The cross-repo link was the only latent 404.The npm deprecation message for
@opensea/stream-jsis being updated to the same pinned URL, for the same reason.Safe to merge and re-archive.
🤖 Generated with Claude Code