[
https://issues.apache.org/jira/browse/MCLEAN-126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17897111#comment-17897111
]
Peter De Maeyer edited comment on MCLEAN-126 at 11/11/24 8:21 AM:
------------------------------------------------------------------
I made a PR with minimal "like for like" replacements, focusing on production
code. There are more opportunities to leverage the {{Path}} API further, for
example leveraging its built-in support for symlinks to replace the
roll-your-own symlink code with, but I left that out of scope of for now.
was (Author: peterdm):
I made a PR with minimal "like for like" replacements, focusing on production
code. There are more opportunities to leverage the `Path` API further, for
example leveraging its built-in support for symlinks to replace the
roll-your-own symlink code with, but I left that out of scope of for now.
> Use Path instead of File paradigm as much as possible
> -----------------------------------------------------
>
> Key: MCLEAN-126
> URL: https://issues.apache.org/jira/browse/MCLEAN-126
> Project: Maven Clean Plugin
> Issue Type: Improvement
> Affects Versions: 3.4.0
> Reporter: Peter De Maeyer
> Priority: Major
>
> The code currently uses a combination of old {{File}} API and old {{Path}}.
> This is suboptimal, as it requires conversions back and forth. It would be
> better if we consolidated on the {{Path}} API as much as possible.
> Unfortunately, the Maven `@Parameter` injection only supports {{File}}
> fields, so we cannot replace it completely yet. Hopefully some day, Maven's
> `@Parameter` injection will also support {{Path}} fields.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
