Skip to content

Remove .prefab file loading functionality and dependent tests#5

Merged
jkebinger merged 1 commit into
mainfrom
remove-prefab-file-loading
Sep 19, 2025
Merged

Remove .prefab file loading functionality and dependent tests#5
jkebinger merged 1 commit into
mainfrom
remove-prefab-file-loading

Conversation

@jkebinger
Copy link
Copy Markdown
Contributor

Summary

  • Removed all .prefab file loading logic from ConfigLoader class
  • Removed prefab_envs variable and related configuration options from Options class
  • Removed all test cases that depended on .prefab file configurations rather than fixing them
  • Cleaned up test files by removing methods that relied on removed functionality
  • Added .claude/ to .gitignore to exclude Claude Code settings

Test plan

  • Verify core functionality tests still pass (non-integration tests should work)
  • Confirm .prefab files are no longer loaded or referenced
  • Ensure removed test methods don't leave broken references

🤖 Generated with Claude Code

- Remove all .prefab file loading logic from ConfigLoader
- Remove prefab_envs variable and related configuration options from Options class
- Remove all test cases that depended on .prefab file configurations
- Clean up test files by removing methods that relied on removed functionality
- Update integration test helpers to use cleaner configurations
- Add .claude/ to .gitignore

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

Co-Authored-By: Claude <noreply@anthropic.com>
@jkebinger jkebinger force-pushed the remove-prefab-file-loading branch from 2640cca to 01ff651 Compare September 19, 2025 22:21
Copy link
Copy Markdown
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 b809718 into main Sep 19, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants