Skip to content

[MWRAPPER-135] Provide a reliable way to determine the Maven Wrapper type #254

Closed

Description

@jira-importer

Tamas Cservenak opened MWRAPPER-135 and commented

Just like MWRAPPER-134 but for type. I completely forgot about this. As this way, tooling can figure out not only version but type (also for humans, just peek at properties).


Issue Links:

  • MWRAPPER-141 Use distribution type of existing Maven Wrapper by default

  • MWRAPPER-134 Provide a reliable way to determine the Maven Wrapper version

Remote Links:

Activity

  1. jira-importer commented on May 17, 2024

    @jira-importer
    Author

    Nils Breunese commented

    Tamas Cservenak I've created a pull request for this feature. Currently the property name is {}distributionType{}, because that is also what it is called in the WrapperMojo class internally. Do you think that is the best name, or should it maybe be {}wrapperType{}?

  2. jira-importer commented on May 17, 2024

    @jira-importer
    Author

    Nils Breunese commented

    Also, if wrapper:wrapper is run on a project that already contains this property in maven-wrapper.properties (for instance when updating Maven Wrapper), would it make sense to use this type by default when type is not explicitly set via {}-Dtype={}? If so, I guess this can also be implemented via a separate ticket.

  3. jira-importer commented on May 17, 2024

    @jira-importer
    Author

    Tamas Cservenak commented

    Thanks Nils Breunese 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions