[MNG-8227] Cleanup after Sisu upgrade
- Dominant language
- Java
- Stars
- 5.3k
- Forks
- 3.1k
- Avg merge
- 20h 42m
- Merged PRs (30d)
- 297
Description
**[Konrad Windszus](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=kwin)** opened **[MNG-8227](https://issues.apache.org/jira/browse/MNG-8227?redirect=false)** and commented
After the upgrade to Sisu 0.9.0 in MNG-7913 some code is no longer necessary:
[PathConverter](https://github.com/apache/maven/blob/acec540547bca289b0579ab0c8f371004b8903f1/maven-core/src/main/java/org/apache/maven/configuration/internal/DefaultBeanConfigurator.java#L175) (shipped with https://github.com/eclipse-sisu/sisu.plexus/commit/8800f2d47ac15e4e059a9dc3bd59e6a39abad49b)
---
**Issue Links:**
- [MNG-7913](https://issues.apache.org/jira/browse/MNG-7913) Upgrade Sisu version to 0.9.0.M2
(_**"is caused by"**_)
Contributor guide
Research direction
Start with maven-core/src/main/java/org/apache/maven/configuration/internal/DefaultBeanConfigurator.java at the PathConverter reference, then read MNG-7913 to understand the Sisu 0.9.0 upgrade. Confirm which code became unnecessary after that upgrade and verify the Maven core build after cleanup.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- build-system
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100