Skip to content
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

Support \r as line seprator within a single workout #24

Closed
mgifos opened this issue Jul 30, 2018 · 1 comment
Closed

Support \r as line seprator within a single workout #24

mgifos opened this issue Jul 30, 2018 · 1 comment

Comments

@mgifos
Copy link
Owner

mgifos commented Jul 30, 2018

It seems that Excell enforces \r, so we'll support it in the future, till then, stick with Google Drive Spreadsheet and probably Open Office.

@mgifos
Copy link
Owner Author

mgifos commented Aug 27, 2018

The issue has been partially resolved, at least from the quick-plan point of view, it support CR or CRLF line endings, but the library it uses to parse CSV still has open issues related to this matter:

Of course in future, we can create our own CSV file parser or finding another one that can parse CR characters even if they are contained within a quoted value without breaking the line.
The workaround for now would be to use Google Spreadsheets or LibreOffice Calc, because both editors force LF character when breaking the line within a cell. Avoid MS Excell for now.

@mgifos mgifos closed this as completed Aug 27, 2018
tonybaines pushed a commit to tonybaines/quick-plan that referenced this issue Jan 1, 2019
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

No branches or pull requests

1 participant