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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gluxhappy
Copy link

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants