brunoborges commented on PR #1599: URL: https://github.com/apache/maven-dependency-plugin/pull/1599#issuecomment-4184438250
## Latest changes (2 new commits) ### 1. Use `-Dgav` only, remove split params (`4ae00111`) Removed separate `@Parameter` fields for `groupId`, `artifactId`, and `version` from both `AddDependencyMojo` and `RemoveDependencyMojo`. The `-Dgav` parameter is now the **only** way to specify dependency coordinates. `-Dscope`, `-Dtype`, `-Dclassifier`, and `-Doptional` remain as separate override parameters (they augment the GAV, not replace it). This addresses the feedback: *"I'd choose between gav or split flavor but I wouldn't start by mixing both"*. ### 2. Rename `DependencyCoordinates` → `DependencyEntry` (`116a4562`) Addresses the feedback: *"scope and optional are not part of coordinates so might be great to find another name"*. --- ### Remaining open design questions 1. **`managed` as enum** — Should `-Dmanaged` be an enum (`YES`/`NO`/`BOTH`) instead of a boolean? Currently it's a boolean that toggles between `<dependencies>` and `<dependencyManagement>`. 2. **`bom` flag** — Should `-Dbom` be removed? Currently it's a convenience flag that sets `scope=import` and `type=pom`. 3. **Version in `validate()`** — Should we require version in validation to avoid "weirdish side effects"? 4. **JSON output** — Should the goals support JSON output for pipeline use? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
