Skip to content

Add additional comment to clarify the minimal supported version of ou…#465

Merged
slawekjaranowski merged 2 commits into
apache:masterfrom
gluxhappy:version_comment_for_json_tree_output
Feb 26, 2025
Merged

Add additional comment to clarify the minimal supported version of ou…#465
slawekjaranowski merged 2 commits into
apache:masterfrom
gluxhappy:version_comment_for_json_tree_output

Conversation

@gluxhappy
Copy link
Copy Markdown
Contributor

@gluxhappy gluxhappy commented Nov 13, 2024

…tputing dependen

Copy link
Copy Markdown
Contributor

@elharo elharo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For myself, I prefer to focus the docs on the current version. I really don't care what was true several versions ago.

Comment thread src/main/java/org/apache/maven/plugins/dependency/tree/TreeMojo.java Outdated
* If specified, this parameter will cause the dependency tree to be written using the specified format. Currently
* supported formats are: <code>text</code> (default), <code>dot</code>, <code>graphml</code>, <code>tgf</code>
* and <code>json</code>.
* and <code>json</code> (since 3.7.0).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know that we need this. It's implicitly the current version being described.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is information in which version such feature was added - for me it is ok

@slawekjaranowski slawekjaranowski added the documentation Improvements or additions to documentation label Feb 26, 2025
@slawekjaranowski slawekjaranowski merged commit 96a660f into apache:master Feb 26, 2025
@github-actions github-actions Bot added this to the next-release milestone Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants