move package to src dir

Author: rmorshea

.github/workflows/test.yml

@@ -11,42 +11,39 @@ on:
     - cron: "0 0 * * *"
 
 jobs:
-  test-linux:
+  coverage:
     runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        python-version: [3.7, 3.8, 3.9]
     steps:
       - uses: actions/checkout@v2
       - uses: nanasess/setup-chromedriver@master
       - uses: actions/setup-node@v2-beta
         with:
           node-version: "12"
-      - name: Use Python ${{ matrix.python-version }}
+      - name: Use Latest Python
         uses: actions/setup-python@v2
         with:
-          python-version: ${{ matrix.python-version }}
+          python-version: 3.9
       - name: Install Python Dependencies
         run: pip install -r requirements/test-run.txt
       - name: Run Tests
         run: |
           nox -s test -- pytest[--headless]
-  test-other-systems:
+  matrix:
     runs-on: ${{ matrix.os }}
     strategy:
-      fail-fast: false
       matrix:
-        os: [ macos-latest, windows-latest ]
+        python-version: [3.7, 3.8, 3.9]
+        os: [ubuntu-latest, macos-latest, windows-latest]
     steps:
       - uses: actions/checkout@v2
       - uses: nanasess/setup-chromedriver@master
       - uses: actions/setup-node@v2-beta
         with:
           node-version: "12"
-      - name: Use Python 3.9
+      - name: Use Python ${{ matrix.python-version }}
         uses: actions/setup-python@v2
         with:
-          python-version: 3.9
+          python-version: ${{ matrix.python-version }}
       - name: Install Python Dependencies
         run: pip install -r requirements/test-run.txt
       - name: Run Tests

MANIFEST.in

@@ -1,10 +1,6 @@
-recursive-include idom/client/app *
-recursive-exclude idom/client/app/node_modules *
-recursive-exclude idom/client/app/web_modules *
-recursive-exclude idom/client/build *
-include idom/py.typed
-include LICENSE
+recursive-include src/idom *
+recursive-exclude src/idom/client/app/node_modules *
+recursive-exclude src/idom/client/app/web_modules *
+recursive-exclude src/idom/client/build *
 include requirements/prod.txt
 include requirements/extras.txt
-include README.md
-include scripts/build.sh

noxfile.py

@@ -87,9 +87,14 @@ def test_python(session: Session) -> None:
     """Run the Python-based test suite"""
     session.env["IDOM_DEBUG_MODE"] = "1"
     session.install("-r", "requirements/test-env.txt")
-    session.install(".[all]")
-    args = ["pytest", "tests"] + get_posargs("pytest", session)
-    session.run(*args)
+
+    pytest_args = get_posargs("pytest", session)
+    if "--no-cov" in pytest_args:
+        session.install(".[all]")
+    else:
+        session.install("-e", ".[all]")
+
+    session.run("pytest", "tests", *pytest_args)
 
 
 @nox.session
@@ -98,14 +103,14 @@ def test_types(session: Session) -> None:
     session.install("-r", "requirements/check-types.txt")
     session.install("-r", "requirements/pkg-deps.txt")
     session.install("-r", "requirements/pkg-extras.txt")
-    session.run("mypy", "--strict", "idom")
+    session.run("mypy", "--strict", "src/idom")
 
 
 @nox.session
 def test_style(session: Session) -> None:
     """Check that style guidelines are being followed"""
     session.install("-r", "requirements/check-style.txt")
-    session.run("flake8", "idom", "tests", "docs")
+    session.run("flake8", "src/idom", "tests", "docs")
     black_default_exclude = r"\.eggs|\.git|\.hg|\.mypy_cache|\.nox|\.tox|\.venv|\.svn|_build|buck-out|build|dist"
     session.run(
         "black",

scripts/__init__.py

@@ -13,14 +13,14 @@ max-line-length = 88
 max-complexity = 18
 select = B,C,E,F,W,T4,B9,N,ROH
 exclude =
-    idom/client/app/node_modules/*
+    src/idom/client/app/node_modules/*
     .eggs/*
     .tox/*
 
 [tool:pytest]
 testpaths = tests
 xfail_strict = True
-addopts = --cov=idom --cov-report term
+addopts = --cov=src/idom --cov-report term
 markers =
     slow: marks tests as slow (deselect with '-m "not slow"')
 
@@ -30,11 +30,11 @@ show_missing = True
 skip_covered = True
 sort = Name
 exclude_lines =
-    pragma: no cover
+    coverage: skip
     \.\.\.
     raise NotImplementedError
 omit =
-    idom/__main__.py
+    src/idom/__main__.py
 
 [build_sphinx]
 all-files = true

setup.cfg

@@ -1,6 +1,5 @@
 from __future__ import print_function
 
-import os
 import pipes
 import shutil
 import subprocess
@@ -9,6 +8,7 @@
 from distutils import log
 from distutils.command.build import build  # type: ignore
 from distutils.command.sdist import sdist  # type: ignore
+from pathlib import Path
 
 from setuptools import find_packages, setup
 from setuptools.command.develop import develop
@@ -25,8 +25,9 @@ def list2cmdline(cmd_list):
 name = "idom"
 
 # basic paths used to gather files
-here = os.path.abspath(os.path.dirname(__file__))
-root = os.path.join(here, name)
+root_dir = Path(__file__).parent
+src_dir = root_dir / "src"
+package_dir = src_dir / name
 
 
 # -----------------------------------------------------------------------------
@@ -37,7 +38,8 @@ def list2cmdline(cmd_list):
 package = {
     "name": name,
     "python_requires": ">=3.7",
-    "packages": find_packages(exclude=["tests*"]),
+    "packages": find_packages(str(src_dir)),
+    "package_dir": {"": "src"},
     "description": "Control the web with Python",
     "author": "Ryan Morshead",
     "author_email": "ryan.morshead@gmail.com",
@@ -76,16 +78,16 @@ def list2cmdline(cmd_list):
 
 
 requirements = []
-with open(os.path.join(here, "requirements", "pkg-deps.txt"), "r") as f:
+with (root_dir / "requirements" / "pkg-deps.txt").open() as f:
     for line in map(str.strip, f):
         if not line.startswith("#"):
             requirements.append(line)
 package["install_requires"] = requirements
 
 _current_extras = []
 extra_requirements = {"all": []}  # type: ignore
-extra_requirements_path = os.path.join(here, "requirements", "pkg-extras.txt")
-with open(extra_requirements_path, "r") as f:
+extra_requirements_path = root_dir / "requirements" / "pkg-extras.txt"
+with extra_requirements_path.open() as f:
     for line in map(str.strip, f):
         if line.startswith("#") and line[1:].strip().startswith("extra="):
             _current_extras = [e.strip() for e in line.split("=", 1)[1].split(",")]
@@ -98,8 +100,9 @@ def list2cmdline(cmd_list):
                 extra_requirements[e].append(line)
             extra_requirements["all"].append(line)
         elif line:
-            msg = "No '# extra=<name>' header before requirements in %r"
-            raise ValueError(msg % extra_requirements_path)
+            raise ValueError(
+                f"No '# extra=<name>' header before requirements in {extra_requirements_path}"
+            )
 package["extras_require"] = extra_requirements
 
 
@@ -108,7 +111,7 @@ def list2cmdline(cmd_list):
 # -----------------------------------------------------------------------------
 
 
-with open(os.path.join(here, "README.md")) as f:
+with (root_dir / "README.md").open() as f:
     long_description = f.read()
 
 package["long_description"] = long_description
@@ -125,7 +128,7 @@ class Command(cls):
         def run(self):
             log.info("Installing Javascript...")
             try:
-                js_dir = os.path.join(root, "client", "app")
+                js_dir = str(package_dir / "client" / "app")
                 for cmd, *args in map(str.split, ["npm install", "npm run build"]):
                     which_cmd = shutil.which(cmd)
                     if which_cmd is None:

setup.py

@@ -3,7 +3,7 @@
 
 try:
     __version__: str = _get_distribution(__name__).version
-except _DistributionNotFound:  # pragma: no cover
+except _DistributionNotFound:  # coverage: skip
     # package is not installed
     __version__ = "0.0.0"
 
@@ -24,7 +24,7 @@
     import htm
     import pyalect
     import tagged
-except ImportError:  # pragma: no cover
+except ImportError:  # coverage: skip
     pass
 else:
     from . import dialect