fix: remove phantom git submodule reference at path "reframe" - #1765
Merged
Conversation
The repo root has had a stray gitlink (mode 160000) at path "reframe" pointing to commit 9022ede, with no corresponding .gitmodules entry to give it a URL. Any tool that clones with submodule init fails on it: this just broke a Netlify deploy ("Error checking out submodules: fatal: No url found for submodule path 'reframe' in .gitmodules") and is the same error seen in a Chromatic Actions run's post-job cleanup a few days ago, previously undiagnosed. Also picks up .gitignore entries (.env*, .netlify) added locally by the Netlify/Vercel CLI setup earlier. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01L5gqyU5QKQtLAza6hL6dXv
✅ Deploy Preview for reframe-os ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
Bot
added
level:beginner
Contributor
👋 Thanks for your PR, @magic-peach!Welcome to Reframe — a browser-based video editor built for everyone 🎬 What happens next
Quick checklist
Useful links
Happy coding! 🎉 |
Contributor
|
Contributor
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
magic-peach
added a commit
that referenced
this pull request
Sep 2, 2026
main picked up the org rename (#1756) and the phantom-submodule fix (#1765) since this branch was created; both landed cleanly. bun.lock had a real conflict (main gained Snyk's dependency, this branch gained the Datadog RUM packages) — resolved by regenerating the lockfile via `bun install` against the merged package.json rather than hand-editing conflict markers. tsc --noEmit, lint, and build all verified clean post-merge. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01L5gqyU5QKQtLAza6hL6dXv
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
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.
Summary
reframe, pointing to commit9022ede4215ab686a829b4bd88a25cf67bdeb248with no corresponding.gitmodulesentry.Error checking out submodules: fatal: No url found for submodule path 'reframe' in .gitmodules. The identical error also appears in a Chromatic Actions run's post-job git cleanup from a few days ago — previously unexplained, now accounted for. Any tool that clones with submodule init enabled hits this..gitignoreentries (.env*,.netlify) added locally by the Netlify/Vercel CLI setup, unrelated but already sitting in the working tree.Test plan
git ls-tree origin/mainno longer shows a160000entry after this mergesreframe-osonce this is onmain🤖 Generated with Claude Code
https://claude.ai/code/session_01L5gqyU5QKQtLAza6hL6dXv