dbcli / litecli

CLI for SQLite Databases with auto-completion and syntax highlighting
https://litecli.com
BSD 3-Clause "New" or "Revised" License
2.09k stars 67 forks source link

test/data/import_data.csv missing from release tarball for 1.3.2 #87

Closed stigtsp closed 7 months ago

stigtsp commented 4 years ago

Hi,

The release tarball is missing import_data.csv (and liteclirc) causing tests to fail.


$ diff -ur litecli-tag-v1.3.2 litecli-tarball-1.3.2
Only in litecli-tag-v1.3.2: CONTRIBUTING.md
Only in litecli-tag-v1.3.2: .coveragerc
Only in litecli-tag-v1.3.2: .github
Only in litecli-tag-v1.3.2: .gitignore
Only in litecli-tarball-1.3.2: litecli.egg-info
Only in litecli-tarball-1.3.2: PKG-INFO
Only in litecli-tag-v1.3.2: .pre-commit-config.yaml
Only in litecli-tag-v1.3.2: screenshots
diff -ur litecli-tag-v1.3.2/setup.cfg litecli-tarball-1.3.2/setup.cfg
--- litecli-tag-v1.3.2/setup.cfg    2020-03-30 17:51:52.845302447 +0200
+++ litecli-tarball-1.3.2/setup.cfg 2020-03-11 18:55:57.000000000 +0100
@@ -3,16 +3,21 @@

 [tool:pytest]
 addopts = --capture=sys
-          --showlocals
-          --doctest-modules
-          --doctest-ignore-import-errors
-          --ignore=setup.py
-          --ignore=litecli/magic.py
-          --ignore=litecli/packages/parseutils.py
-          --ignore=test/features
+   --showlocals
+   --doctest-modules
+   --doctest-ignore-import-errors
+   --ignore=setup.py
+   --ignore=litecli/magic.py
+   --ignore=litecli/packages/parseutils.py
+   --ignore=test/features

 [pep8]
 rev = master
 docformatter = True
 diff = True
 error-status = True
+
+[egg_info]
+tag_build = 
+tag_date = 0
+
Only in litecli-tag-v1.3.2/tests: data
Only in litecli-tag-v1.3.2/tests: liteclirc
Only in litecli-tag-v1.3.2: TODO
Only in litecli-tag-v1.3.2: .travis.yml
amjith commented 4 years ago

I'm surprised tests are included in the release tarball.

I'm curious why you run the tests from the tarball and not from a git clone? I'm trying to understand the use case so I can better serve the need.

amjith commented 4 years ago

Duh, for making nixOS packages, I presume. I'll include the csv file.