ascheman commented on issue #11612:
URL: https://github.com/apache/maven/issues/11612#issuecomment-3746350498

   ## Clarification: Additional Acceptance Criteria
   
   Based on @desruisseaux's [review 
feedback](https://github.com/apache/maven/issues/11612#issuecomment-3732564412):
   
   > * A project can be either fully modular or fully classical, but not a mix 
of both because the compiler plugin will not know how to handle that. I suggest 
to raise an error when such mix is detected.
   > * In a modular project, I suggest to unconditionally ignore 
`<sourceDirectory>` and `<testSourceDirectory>` because it is not clear how to 
dispatch the content of those directories between the different modules. A 
warning should be emitted if these properties are set.
   
   Added explicit acceptance criteria:
   - **AC6**: ERROR for mixed modular/classic sources within `<sources>` 
(validates R2)
   - **AC7**: WARNING for legacy directory usage in modular projects (validates 
R4)
   
   These behaviors were already implied by validation rules R2 and R4 in the 
table, but are now explicit acceptance criteria.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to