Conversation
|
Hi, IMO: This functionality can be tailored by users as needed. |
fastexcel-core/src/main/java/cn/idev/excel/write/merge/DynamicMergeStrategy.java
Outdated
Show resolved
Hide resolved
fastexcel-core/src/main/java/cn/idev/excel/write/merge/DynamicMergeStrategy.java
Outdated
Show resolved
Hide resolved
fastexcel-core/src/main/java/cn/idev/excel/write/merge/DynamicMergeStrategy.java
Outdated
Show resolved
Hide resolved
fastexcel-test/src/test/java/cn/idev/excel/test/demo/write/WriteTest.java
Outdated
Show resolved
Hide resolved
|
Your suggestion is very good, I will revise it and submit it again |
|
Hi, maybe I wasn't clear enough. Kindly delete the author information and refrain from including any personal developer specifics :) |
|
Update: In the code review process. |
wangmiscoding
left a comment
There was a problem hiding this comment.
It seems that the code is not formatted, and the character spacing is not consistent
|
Hi, @long-chin We should resolve conflicts first, and please modify the commit message to better comply with the commit guidelines,and use English for commit messages. |
…cified column (cherry picked from commit 8cceadf)
No description provided.