Declare javax.inject dependency by elharo · Pull Request #197 · apache/maven-pmd-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 1 commit into

Apr 23, 2025

Conversation

@elharo

@elharo

@elharo elharo marked this pull request as ready for review

April 23, 2025 11:10

@Bukama

Thought we do DI via sisu - however in the build sisu-inject-plexusis excluded?

@elharo

We have so many variants of DI, and I have no idea which is used where at this point.

https://xkcd.com/927/

olamy
javax.inject
javax.inject
1

Choose a reason for hiding this comment

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

shouldn't be provided scope?

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