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
  • !1089

Fix csvformat -U2 while not impeding performance in other cases

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Loïc Corbasson requested to merge github/fork/lcorbasson/master into master Sep 04, 2020
  • Overview 3
  • Commits 7
  • Pipelines 1
  • Changes 2

Rationale:

  • csvformat -U2 doesn't work -- it's a bug.
  • csvformat -U2 without -u2 means that we'll have to detect which columns are numeric, to convert the input (string) values to output (decimal) values.
  • We don't want to change anything to the other columns.
  • We need to keep things as quick as possible.

This is an attempt at solving this bug while responding to the above constraints. Feel free to comment or hack it!

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/lcorbasson/master