Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • C csvkit
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 61
    • Issues 61
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 4
    • Merge requests 4
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • wireservice
  • csvkit
  • Merge requests
  • !212

Feature/add escape lf option

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Administrator requested to merge github/fork/billwanjohi/feature/add_escape_lf_option into master May 24, 2013
  • Overview 0
  • Commits 6
  • Pipelines 0
  • Changes 13

Created by: billwanjohi

Excel spreadsheets often have cell values that span multiple lines. The CSV MIME Type only requires that you wrap such values in double quotes. This is sufficient for many CSV parsers, but breaks others.

I added an option to in2csv ('--escape_lf') that precedes every line feed character with a backslash, thus escaping it and preventing "dumber" parsers from treating the partial value as a separate row.

I tried to be as complete as possible, adding example data, tests, and a mention in the docs.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/billwanjohi/feature/add_escape_lf_option