Skip to content

Replace readonly @Parameter annotations for components with @Inject#5224

Open
HannesWell wants to merge 1 commit intoeclipse-tycho:mainfrom
HannesWell:inject-components
Open

Replace readonly @Parameter annotations for components with @Inject#5224
HannesWell wants to merge 1 commit intoeclipse-tycho:mainfrom
HannesWell:inject-components

Conversation

@HannesWell
Copy link
Copy Markdown
Member

This modernizes common old ways of configurations like

	@Parameter(property = "session", required = true)
	private MavenSession session;
	@Parameter(property = "project", readonly = true)
	private MavenProject project;

to just

	@Inject
	private MavenSession session;
	@Inject
	private MavenProject project;

which should also remove these parameters from the Mojo's help page, which can actually not be configured.

And migrate some plain @Component annotations by @Inject.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 8, 2025

Test Results

1 008 files  1 008 suites   5h 10m 46s ⏱️
1 293 tests 1 269 ✅ 20 💤 1 ❌ 3 🔥
3 879 runs  3 805 ✅ 64 💤 1 ❌ 9 🔥

For more details on these failures and errors, see this check.

Results for commit 57fd179.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant