Commented code in MojoExtension.java
- Dominant language
- Java
- Stars
- 5.3k
- Forks
- 3.1k
- Avg merge
- 20h 42m
- Merged PRs (30d)
- 297
Description
### Affected version
HEAD
### Bug description
figure out what to do about this, probably delete it
```
if (parameterConfiguration != null) {
Map attributes = new HashMap<>(parameterConfiguration.getAttributes());
// if (isEmpty(parameterConfiguration.getAttribute("implementation"))
// && !isEmpty(parameter.getImplementation())) {
// attributes.put("implementation", parameter.getImplementation());
// }
parameterConfiguration = XmlNode.newInstance(
parameter.getName(),
parameterConfiguration.getValue(),
attributes,
parameterConfiguration.getChildren(),
parameterConfiguration.getInputLocation());
children.add(parameterConfiguration);
}
}
```
Contributor guide
Research direction
Open MojoExtension.java and inspect the commented-out block in the parameter configuration handling shown in the issue. Determine whether it is obsolete, then remove it if appropriate and verify that the Maven build remains unaffected.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- build-system
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 78/100