slawekjaranowski commented on PR #151:
URL: 
https://github.com/apache/maven-plugin-tools/pull/151#issuecomment-1284431737

   > Looks good (apart from the very minor indentation issue). Some of it had 
already been integrated in the v4 api through 
[gnodet/maven@6317b8c](https://github.com/gnodet/maven/commit/6317b8c0ad468a4abdee47531d461755ff051d0a),
 but both `@Parameter` and `@Component` annotations were actually modified. 
Does it make sense or should I revert the changes on `@Component` ?
   
   We can add support for `@Compoent` with method in next step.
   I must check if old `@Compoent` can be injected by method - maybe you know 
it.
   
   I hope we can also add support for old and new annotation together. 
   As soon Maven 4 alpha will be published we will have open way for it.


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