fix issue#98 : adding column_table_limit feature #106
Conversation
|
|
||
| The column limit of one line for table. Default value is same as column_limit value. | ||
|
|
||
| ```lua |
There was a problem hiding this comment.
Change the examples to something like:
column_table_limit: column_limit
[...]
column_table_limit: 20
[...]
docs/Style-Config.md
Outdated
|
|
||
| type: int, default: column_limit | ||
|
|
||
| The column limit of one line for table. Default value is same as column_limit value. |
There was a problem hiding this comment.
| The column limit of one line for table. Default value is same as column_limit value. | |
| The column limit of each line of a table. Default value the same as column_limit. |
|
@karanankit01 squash the commits into 2. Fixes for commits not yet merged should not go to |
| chop_down_table: false | ||
| chop_down_kv_table: true | ||
| table_sep: "," | ||
| column_table_limit: column_limit |
There was a problem hiding this comment.
The default value of column_table_limit is not a reference to column_limit .
Consider this config file:
column_limit: 120The value of column_table_limit is 80, not 120.
There was a problem hiding this comment.
ok, I figured it out.
I fixed it by checking the config file and updating its value with column_limit .
karanankit01
force-pushed
the
cov-test
branch
4 times, most recently
from
4a649b2 to
00026e8
Compare
src/Config.cpp
Outdated
| if (node_[key]) { | ||
| node_[key] = kv.second; | ||
| } | ||
| if(key == CTL) { |
There was a problem hiding this comment.
this code is not formatted correctly.
use clang-format.
There was a problem hiding this comment.
Done.
Thanks, I will mind it from now.
3 participants
by default the column_table_limit is same as column_limit.
User can set the value of column_table_limit in type int.
as discussed in issue#98