Skip to content

Commit

Permalink
Fix storey distribution - don't include tests/integration in pack…
Browse files Browse the repository at this point in the history
…ages (#534)

* Remove an unused Dockerfile

* Update package Python support

* Update .gitignore

* Change template to .gitignore

* Remove `.dockerignore` - no Docker in storey

* Add egg-info to .gitignore

* Add Lua comment on lupa

* Add a small type hint

* Add dist to .gitignore

* Add .DS_Store to .gitignore

A Mac generated file

* Remove .dockerignore from MANIFEST.in file

* Include only storey package in the distribution

* Add __pycache__/ to gitignore

* Fix sub-packages discovery

* Add .python-version global file

For pyenv/uv

* Update .gitignore

* Revert "Add .python-version global file"

This reverts commit 6ad0cee.

* Add `docs/_build/` and vscode to .gitignore
  • Loading branch information
jond01 authored Sep 8, 2024
1 parent f89c2ef commit d36945a
Show file tree
Hide file tree
Showing 7 changed files with 36 additions and 22 deletions.
3 changes: 0 additions & 3 deletions .dockerignore

This file was deleted.

28 changes: 28 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
__pycache__/
*.py[cod]

*.egg-info/
dist/

venv/

*.coverage
coverage_reports/

.python-version

docs/_build/

test.db
bench-results.json

.idea

.vscode/*
!.vscode/settings.json
!.vscode/tasks.json
!.vscode/launch.json
!.vscode/extensions.json
*.code-workspace

.DS_Store
5 changes: 0 additions & 5 deletions .gitignore.template

This file was deleted.

7 changes: 0 additions & 7 deletions Dockerfile.test

This file was deleted.

1 change: 0 additions & 1 deletion MANIFEST.in
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
include .dockerignore
include LICENSE
include README.md
include requirements.txt dev-requirements.txt
Expand Down
2 changes: 2 additions & 0 deletions dev-requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ isort~=5.7
pytest~=5.4.3
coverage~=7.5
pytest-benchmark~=3.2.3
# Note: you might need a Lua installation to install lupa on Mac.
# brew install lua
lupa~=1.13
fakeredis~=1.9
redis~=4.3
Expand Down
12 changes: 6 additions & 6 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,17 @@
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#

from setuptools import find_packages, setup


def version():
def version() -> str:
with open("storey/__init__.py") as fp:
for line in fp:
if line.startswith("__version__"):
_, version = line.split("=")
return version.replace('"', "").strip()
raise ValueError("Could not find package version")


def load_deps(file_name):
Expand Down Expand Up @@ -58,7 +59,8 @@ def load_deps(file_name):
author_email="yaronh@iguazio.com",
license="Apache",
url="https://github.com/mlrun/storey",
packages=find_packages(),
packages=find_packages(include=["storey*"]),
python_requires=">=3.9",
install_requires=install_requires,
extras_require=extras_require,
classifiers=[
Expand All @@ -69,9 +71,7 @@ def load_deps(file_name):
"Operating System :: Microsoft :: Windows",
"Operating System :: MacOS",
"Programming Language :: Python :: 3",
"Programming Language :: Python :: 3.6",
"Programming Language :: Python :: 3.7",
"Programming Language :: Python :: 3.8",
"Programming Language :: Python :: 3.9",
"Programming Language :: Python",
"Topic :: Software Development :: Libraries :: Python Modules",
"Topic :: Software Development :: Libraries",
Expand Down

0 comments on commit d36945a

Please sign in to comment.