CSV stringify

[tshemsedinov]

We need to implement analog of https://www.npmjs.com/package/csv-stringify but we don't need everything, only callback-api https://csv.js.org/stringify/api/#callback-api

stringify(data, (err, str) => { });

[tshemsedinov]

You can implement this task together @yehorbk @IegorT it is not so simple, csv specification is a complex serialization format, but we can separate this task into subtasks:

• API definition and API stub
• Serialization implementation
• Code review
• Serialization optimization
• Unit tests
• Examples and docs

Who will be first, please write here, fork and try to start with first subtask. After that second one will wait to review your work with me and merge pull request. Only after that second contributor may start implementation. In parallel first contributor may start implementing unittests. Then you will review code of each other and merge your code into internal branch of this repo so we will test it together.

[yehorbk]

I am ready to work. What I need to fork? Original csv-stringify repo or metarhia/common?

[yehorbk]

I am ready to work. What I need to fork? Original csv-stringify repo or metarhia/common?

I've forked metarhia/common and realized that I need to read the documentation of metatests.

[yehorbk]

I created a branch csv-stringify and added module with stub and tests with basic metatests structure there. Also module connected to library in common.js.
Am I need to write something to README.md?

[tshemsedinov]

@IegorT are you in?
We have a stub in csv branch: https://github.com/metarhia/common/tree/csv
So you can clone it and implement while @yehorbk will start unittests.

[IegorT]

Hi, I've done a part of work, and made PR, but could assign to this issue. #324