gnodet commented on issue #8879:
URL: https://github.com/apache/maven/issues/8879#issuecomment-5388690347

   Looking at the current state of `plugin.mdo`, the removal of 
`requiresReports` from the 1.1.0 model is **already done**. The field is 
defined with bare `<version>1.0.0</version>` and an explicit comment `<!-- no 
longer part of 1.1.0 -->`, so Modello already excludes it from the 1.1.0 and 
2.0.0 generated code. It has no effect on the built artifact.
   
   The field definition in the `.mdo` file is purely historical documentation 
of the 1.0.0 schema (similar to how `Requirement`, `Dependency`, and other 
legacy fields/classes are kept with `1.0.0/1.1.0` version ranges).
   
   If there is still work to be done here, it would be about the **runtime 
behavior** — e.g., ensuring Maven no longer reads or processes this field when 
encountering old 1.0.0 plugin descriptors — rather than simply deleting the 
field definition from the `.mdo` model. Closed PR #12755 which only removed the 
model definition without addressing runtime handling.


-- 
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