hboutemy commented on code in PR #1147:
URL: https://github.com/apache/maven/pull/1147#discussion_r1220964499


##########
api/maven-api-model/src/main/mdo/maven.mdo:
##########
@@ -2646,6 +2646,15 @@
             ]]>
           </description>
         </field>
+        <field>
+          <name>priority</name>
+          <version>4.2.0+</version>
+          <type>int</type>
+          <description>The priority of this execution compared to other 
executions which are bound to the same phase.
+            Executions derived from the default lifecycle have a negative 
priority by default so that they are executed
+            before any custom plugin executions.

Review Comment:
   nice description
   idea: IIRC, priority is set to -1 for lifecycle bindings, where default 
priority is 0
   while introducing this priority, using -1000 instead of -1 would permit 
users to insert between lifecycle binding and default 0 (and would start to 
clarify why priority vales is not ideal, but just an easy opportunistic feature 
to use only when necessary)
   
   note for self: I also need to check that when bindings have been defined 
with default priority, we can in a child pom just override the priority by just 
putting execution id



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