Skip to content

Conversation

@echonesis
Copy link
Contributor

Purpose of the pull request

Close: Task 3.3 in #443

What's changed?

Checklist

  • I have written the necessary doc or comment.

@delei
Copy link
Member

delei commented Aug 1, 2025

Thanks @echonesis
I have proofread part of the content and made some minor adjustments. Please refer to PR #460 for optimisation.

Please rename the file directly to csv.md.

@delei delei self-requested a review August 1, 2025 06:24
FastExcel.write(csvFile, DemoData.class)
.csv()
.delimiter(CsvConstant.UNICODE_EMPTY)
.doWrite(List<DemoData>);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use data() instead of List<DemoData>

#### 代码示例
```java
@Test
public void writeCsvWithEscape() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

modify the method name to escapeDemo()

Copy link
Member

@delei delei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@psxjoy psxjoy merged commit 8a887f1 into apache:main Aug 5, 2025
5 checks passed
@echonesis echonesis deleted the doc-write-csv branch August 12, 2025 09:41
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