Skip to content

Feat: Expose scope related bits via scope manager#1862

Merged
cstamas merged 2 commits into
apache:masterfrom
cstamas:scope-ops
May 11, 2026
Merged

Feat: Expose scope related bits via scope manager#1862
cstamas merged 2 commits into
apache:masterfrom
cstamas:scope-ops

Conversation

@cstamas
Copy link
Copy Markdown
Member

@cstamas cstamas commented May 10, 2026

When Resolver 2.x is used with ScopeManager, the scope semantics cannot be taken for granted as it was in Maven 3.9 and before, as in this case, scope manager configuration is the one who defines them, and their semantics as well.

Hence, any scope related operation, like filtering and selecting (during collection) must be done in with scope manager prepared filter or selector.

This PR fixes an issue never addressed in Resolver 2.x so far: it deprecated all the old "utils" that had wired-in scope semantics from Maven 3.9 and before times, but did not provide functionally equivalent replacement.

When Resolver 2.x is used with ScopeManager, the scope semantics cannot be
taken for granted as it was in Maven 3.9 and before, as in this case
scope manager configuration is the one who defines them, and their semantics
as well.

Hence, any scope related operation, like filtering and selecting (during collection)
must be done in with scope manager prepared filter or selector.
@cstamas cstamas added this to the 2.0.18 milestone May 10, 2026
@cstamas cstamas requested a review from slawekjaranowski May 10, 2026 12:59
@cstamas cstamas self-assigned this May 10, 2026
@cstamas cstamas added the enhancement New feature or request label May 10, 2026
@cstamas cstamas marked this pull request as ready for review May 11, 2026 10:06
@cstamas cstamas merged commit 0ba854d into apache:master May 11, 2026
14 checks passed
@cstamas cstamas deleted the scope-ops branch May 11, 2026 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant