-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
breaking changeWill break existing functionalityWill break existing functionalitycomponent: configConfiguration systemConfiguration systemenhancementNew feature or requestNew feature or requestpriority: highHigh priority features/fixeHigh priority features/fixerelease: v1.1.0Targeted for v1.1.0Targeted for v1.1.0
Description
🎯 Issue Summary
Move interface path from Application/Interfaces
to Domain/Contracts
to comply with Clean Architecture principles where contracts belong in the Domain layer.
💥 Breaking Change Details
Current Behavior (v1.0.x)
// config/config.php
'paths' => [
'interface' => 'Application/Interfaces',
]
// Generated files
app/Modules/Blog/Application/Interfaces/PostRepositoryInterface.php
// Namespace: App\Modules\Blog\Application\Interfaces
New Behavior (v1.1.x)
// config/config.php
'paths' => [
'interface' => 'Domain/Contracts',
'contract' => 'Domain/Contracts', // Alias for clarity
]
// Generated files
app/Modules/Blog/Domain/Contracts/PostRepositoryInterface.php
// Namespace: App\Modules\Blog\Domain\Contracts
Test Updates
Update tests to be in compliance with the new configuration
Metadata
Metadata
Assignees
Labels
breaking changeWill break existing functionalityWill break existing functionalitycomponent: configConfiguration systemConfiguration systemenhancementNew feature or requestNew feature or requestpriority: highHigh priority features/fixeHigh priority features/fixerelease: v1.1.0Targeted for v1.1.0Targeted for v1.1.0
Projects
Status
Ready