Attention is currently required from: daniel, dexter, laforge.

View Change

3 comments:

• File contrib/csv-to-pgsql.py:

  • Patch Set #5, Line 38: table_name

    I am not sure, yes we could do the .lower() here. I think its ok to keep the . […]

    Consistency with what? Now that you have `.lower()` here, I see no point doing this below.

• File contrib/csv-to-pgsql.py:

  • Patch Set #7, Line 80: self.table

    Shouldn't this be `table_names`?

  • Patch Set #7, Line 117: str(self.table.lower()))

    ```suggestion
            log.info("New database table created: %s", self.table)
    ```

To view, visit change 41508. To unsubscribe, or for help writing mail filters, visit settings.