-
Notifications
You must be signed in to change notification settings - Fork 316
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
chore(docs): explain -maxdet #1035
base: master
Are you sure you want to change the base?
Conversation
@@ -99,7 +99,7 @@ Html report related | |||
Path to custom mustache template/css directory, needs to contain full set of templates. | |||
Bundled templates can be found in jar '/layout' and can be extracted with jar tool or any zip capable tool. | |||
[-maxdet number] | |||
Limit for when tables should be shown with details. | |||
Threshold for details in diagrams, increase if you still want details, checked against sum of tables and views. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make more concise.
Maybe something like "Maximum number of columns displayed in the entity-relationship diagram"
Possibly add default value and default behaviour
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since your example is incorrect, there's something wrong with the text or the "option"
Show table/view details(all columns) as long as the sum(tables + views) are lower than this assigned value. Default is 400 i think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
boolean showDetailedTables = tables.size() <= config.getMaxDetailedTables();
Only issue is that tables is tables and views.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, this is definitely not obvious at all. In addition to the question of what details
is, what diagrams
are we talking about? Compact, large, one degree, two degrees?
I think I need a better understanding of the scenario where we use this, before I can suggest improvements.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
boolean showDetailedTables = tables.size() <= config.getMaxDetailedTables(); |
The parameter is also described further down in the file. |
Hmm, i think the one I changed should be removed. It's related to diagrams not directly to html report |
fixes #957