Skip to content

Exclude frontpage schema from GeneratedAdventureWorks#154

Merged
oyvindberg merged 1 commit intomainfrom
exclude-frontpage-schema
Jul 23, 2025
Merged

Exclude frontpage schema from GeneratedAdventureWorks#154
oyvindberg merged 1 commit intomainfrom
exclude-frontpage-schema

Conversation

@oyvindberg
Copy link
Copy Markdown
Collaborator

Summary

Exclude the frontpage schema from the main AdventureWorks code generation script to keep it separate from the test suite.

Why this change?

The frontpage schema was added in PR #151 for website documentation examples. However, it was being included in the main test suite generation, which:

  • Added unnecessary code to the test projects (350+ files)
  • Required implementing additional methods in DomainInsert objects
  • Mixed documentation-specific code with test code

Changes

Modified the selector in GeneratedAdventureWorks.scala:

// Before
val selector = Selector.ExcludePostgresInternal

// After  
val selector = Selector.ExcludePostgresInternal and \!Selector.schemas("frontpage")

This ensures the frontpage schema is excluded from code generation while still being available in the database for separate documentation generation scripts.

Test Results

✅ All tests pass successfully with this change

Related

🤖 Generated with Claude Code

- Modified selector to exclude frontpage schema using: Selector.ExcludePostgresInternal and \!Selector.schemas("frontpage")
- This prevents frontpage entities from being generated during normal adventureworks code generation
- The frontpage schema is intended for documentation examples only, not for the main test suite
- All tests pass successfully with this change

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

Co-Authored-By: Claude <noreply@anthropic.com>
@oyvindberg oyvindberg merged commit 32c76e8 into main Jul 23, 2025
8 checks passed
@oyvindberg oyvindberg deleted the exclude-frontpage-schema branch July 28, 2025 00:36
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.

1 participant