Rename plugin folder to MCPForUnity#303
Merged
Merged
Conversation
This is to close #284
Contributor
|
Important Review skippedMore than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review. 55 files out of 196 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits. You can disable this status message by setting the ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Hopefully fix the CI failures
It works! This reverts commit 8c6eaaa.
choej2303
pushed a commit
to choej2303/unity-mcp-ide
that referenced
this pull request
Dec 13, 2025
* Copy UnityMcpBridge into a new MCPForUnity folder This is to close CoplayDev#284 * refactor: rename UnityMcpBridge directory to MCPForUnity in docs * chore: rename UnityMcpBridge directory to MCPForUnity across workflow files * chore: rename UnityMcpBridge directory to MCPForUnity across all files * refactor: update import paths from UnityMcpBridge to MCPForUnity across test files * fix: update module import paths to use MCPForUnity instead of UnityMcpBridge * chore: update unity-mcp package path to MCPForUnity directory * feat: add OneTimeSetUp to initialize CommandRegistry before tests run Hopefully fix the CI failures * Apply recent fix to new folder * Temporarily trigger tests to see if CI works * Revert "Temporarily trigger tests to see if CI works" It works! This reverts commit 8c6eaaa.
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.
The Asset Store needs our plugin folder name to match our asset name. I copied the folder so it doesn't break existing installations.