home / github / issue_comments

Menu
  • Search all tables
  • GraphQL API

issue_comments: 1006232013

This data as json

html_url issue_url id node_id user created_at updated_at author_association body reactions issue performed_via_github_app
https://github.com/simonw/sqlite-utils/pull/361#issuecomment-1006232013 https://api.github.com/repos/simonw/sqlite-utils/issues/361 1006232013 IC_kwDOCGYnMM47-eHN 9599 2022-01-06T02:21:35Z 2022-01-06T02:21:35Z OWNER

I'm having second thoughts about this bit:

Your Python code will be passed a "row" variable representing the imported row, and can return a modified row.

If you are using --lines your code will be passed a "line" variable, and for --all an "all" variable.

The code in question is this:

https://github.com/simonw/sqlite-utils/blob/500a35ad4d91c8a6232134ce9406efec11bedff8/sqlite_utils/utils.py#L296-L303

Do I really want to add the complexity of supporting different variable names there? I think always using value might be better.

Except... value made sense for the existing sqlite-utils convert command where you are running a conversion function against the value for the column in the current row - is it confusing if applied to lines or documents or all?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
1094890366  
Powered by Datasette · Queries took 0.874ms · About: github-to-sqlite