Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged

Conversation

gluxhappy
Copy link
Contributor

@gluxhappy gluxhappy commented Nov 13, 2024

…tputing dependen

Copy link
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.

@@ -108,7 +108,7 @@ public class TreeMojo extends AbstractMojo {
/**
* 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
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
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
20 checks passed
@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