[prev in list] [next in list] [prev in thread] [next in thread] 

List:       maven-dev
Subject:    =?utf-8?q?Re=3A_=5BPR=5D_=5BMNG-8015=5D_Control_the_type_of_path_where_each_?= =?utf-8?q?dependency_
From:       gnodet_(via_GitHub) <git () apache ! org>
Date:       2024-01-31 22:37:15
Message-ID: PR_kwDOAAMmk85kAK0s-4994e5b9-0b0e-470a-8065-af0466db3bd8 () gitbox ! apache ! org
[Download RAW message or body]


gnodet commented on PR #1378:
URL: https://github.com/apache/maven/pull/1378#issuecomment-1920102981

   @desruisseaux I've pushed a small rework of this PR on top of #1391 on my fork.
   The squashed and rebased commit can be found at \
https://github.com/gnodet/maven/commit/c535260586d7eb34c56e836b8385f97ee6c3b9a0  The \
branch merged fork is https://github.com/gnodet/maven/commits/explicit-module-path  
   The main change is that I removed the `DependenciesProperties` in the other PR, so \
there are some adjustments which I'm not completely sure are working.... But the idea \
is that `Type` is extensible.  I think `PathType` should be extensible too, but I \
could not find any location where that would be needed (apart when creating custom \
`Type`s)....  


-- 
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: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic