Switch to Guice constructor injection by elharo · Pull Request #266 · apache/maven-invoker-plugin (original) (raw)

Skip to content

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Merged

elharo

merged 6 commits into

Dec 24, 2024

Conversation

@elharo

@elharo

@elharo elharo changed the titleSwitch to Guice constrcutor injection Switch to Guice constructor injection

Dec 6, 2024

@elharo elharo marked this pull request as ready for review

December 7, 2024 13:03

@elharo

gnodet

gnodet previously requested changes Dec 7, 2024

javax.inject

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a duplicate dependency and Maven 4 fails the build.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@elharo

slawekjaranowski

slachiewicz

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

Labels