Skip to content

Conversation

@jkebinger
Copy link
Contributor

Summary

  • Update prefab-common submodule to point to the merged commit on main
  • Fixes the build failure where TypeScript couldn't find the prefab-common module files

🤖 Generated with Claude Code

Point to the merged commit on main instead of the feature branch.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@jkebinger jkebinger requested a review from a team as a code owner January 5, 2026 21:20
The publish and manual release workflows were failing because the
prefab-common submodule wasn't being checked out. Added submodules: recursive
to the checkout steps.

Replaced the external oclif npmPublish workflow with a custom one that
properly handles submodules.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Contributor

@jdwyah jdwyah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@jkebinger jkebinger merged commit a24d63b into main Jan 5, 2026
6 checks passed
@jkebinger jkebinger deleted the fix-submodule-ref branch January 5, 2026 21:30
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.

3 participants