Skip to content

[Backport 4.0.x] Sync DefaultTypeProvider and dependency-types docs across impl and compat - #12326

Merged
gnodet merged 1 commit into
maven-4.0.xfrom
cherry-pick-12134-to-4.0.x
Jun 19, 2026
Merged

gnodet merged 1 commit into
maven-4.0.xfrom
cherry-pick-12134-to-4.0.x

Conversation

@gnodet

@gnodet gnodet commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Backport of #12134 from master to maven-4.0.x.

…mpat (#12134)

* Sync DefaultTypeProvider and dependency-types.apt across impl and compat

The two DefaultTypeProvider copies (impl/maven-impl and
compat/maven-resolver-provider) had drifted out of sync:
- fatjar type was missing from the impl provider
- test-java-source type was missing from the compat provider
- CLASSPATH_PROCESSOR and MODULAR_PROCESSOR ordering was inconsistent

Both dependency-types.apt files also had copy-paste errors from the
test-jar row (bogus "tests" classifier on modular-jar, classpath-jar,
and fatjar) and were missing fatjar's includesDependencies=true flag.
Added the missing processor types and test-java-source to both docs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Add APT consistency tests for DefaultTypeProvider in impl and compat

Validates dependency-types.apt against DefaultTypeProvider, checking
classifier, extension, language, path types, and includesDependencies.
Also verifies all provider types are documented. Mirrors the existing
ArtifactHandlerTest pattern.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
@gnodet gnodet added this to the 4.0.0 milestone Jun 19, 2026
@gnodet gnodet added the bug Something isn't working label Jun 19, 2026
@gnodet
gnodet merged commit ba4f065 into maven-4.0.x Jun 19, 2026
19 of 21 checks passed
@gnodet
gnodet deleted the cherry-pick-12134-to-4.0.x branch June 19, 2026 08:06
@gnodet gnodet modified the milestones: 4.0.0, 4.0.0-rc-6 Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant