fix: Number takes priority over date-time, if not using --datetime-fo…

…rmat, closes #1270
wireservice · Feb 6, 2025 · c327a1b · c327a1b
1 parent 0095bed
commit c327a1b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -7,6 +7,7 @@ Unreleased
 -  feat: :doc:`/scripts/sql2csv` adds a :code:`--execution-option` option.
 -  feat: :doc:`/scripts/sql2csv` uses the ``stream_results=True`` execution option, by default, to not load all data into memory at once.
 -  fix: :doc:`/scripts/csvsql` uses a default value of 1 for the :code:`--min-col-len` and :code:`--col-len-multiplier` options.
+-  fix: For type inference, number takes priority over date-time, if not using the :code:`--datetime-format` option.
 
 2.0.1 - July 12, 2024
 ---------------------

diff --git a/csvkit/cli.py b/csvkit/cli.py
@@ -368,8 +368,10 @@ def get_column_types(self):
                 ]
 
                 # In order to parse dates like "20010101".
-                if self.args.date_format or self.args.datetime_format:
+                if self.args.datetime_format:
                     types.insert(-1, number_type)
+                elif self.args.date_format:
+                    types.insert(-2, number_type)
                 else:
                     types.insert(1, number_type)