• 
      

    Add tox for testing, update setuptools deps, and fix <= 3.9 testing.

    Review Request #15109 — Created June 10, 2026 and updated

    Information

    buildthings
    master

    Reviewers

    To improve testing going forward, this change introduces support for
    tox. It's set up to test with 3.9 through 3.14 (but not 3.8, as the
    modern license files in our pyproject.toml aren't compatible with the
    latest setuptools for 3.8).

    Some test-specific issues were caught with tthis on 3.9 and older. We
    were asserting on Distribution.name, which wasn't available. This
    check isn't really necessary, and has been removed.

    To ensure consistency with testing, setuptools has also been updated
    to the latest versions available.

    Ran tox. All tests pass.

    Summary ID
    Add tox for testing, update setuptools deps, and fix <= 3.9 testing.
    To improve testing going forward, this change introduces support for `tox`. It's set up to test with 3.9 through 3.14 (but not 3.8, as the modern license files in our `pyproject.toml` aren't compatible with the latest `setuptools` for 3.8). Some test-specific issues were caught with tthis on 3.9 and older. We were asserting on `Distribution.name`, which wasn't available. This check isn't really necessary, and has been removed. To ensure consistency with testing, `setuptools` has also been updated to the latest versions available.
    d051266bdc93bcbedd326d941db97c715699afb2
    Description From Last Updated

    Interdiff seems empty?

    david david

    skip_dist isn't a thing in the tox documentation. I think maybe you were thinking of skipsdist, but we probably shouldn't …

    david david

    The preferred way to do this is now package = "editable"

    david david
    david
    1. 
        
    2. tox.toml (Diff revision 1)
       
       
      Show all issues

      skip_dist isn't a thing in the tox documentation. I think maybe you were thinking of skipsdist, but we probably shouldn't be using that.

    3. tox.toml (Diff revision 1)
       
       
      Show all issues

      The preferred way to do this is now package = "editable"

    4. 
        
    chipx86
    Review request changed
    Change Summary:

    Updated tox.toml to use the correct options.

    Commits:
    Summary ID
    Add tox for testing, update setuptools deps, and fix <= 3.9 testing.
    To improve testing going forward, this change introduces support for `tox`. It's set up to test with 3.9 through 3.14 (but not 3.8, as the modern license files in our `pyproject.toml` aren't compatible with the latest `setuptools` for 3.8). Some test-specific issues were caught with tthis on 3.9 and older. We were asserting on `Distribution.name`, which wasn't available. This check isn't really necessary, and has been removed. To ensure consistency with testing, `setuptools` has also been updated to the latest versions available.
    4f4677266401fea7be856b490c5cbee0e4fbd96d
    Add tox for testing, update setuptools deps, and fix <= 3.9 testing.
    To improve testing going forward, this change introduces support for `tox`. It's set up to test with 3.9 through 3.14 (but not 3.8, as the modern license files in our `pyproject.toml` aren't compatible with the latest `setuptools` for 3.8). Some test-specific issues were caught with tthis on 3.9 and older. We were asserting on `Distribution.name`, which wasn't available. This check isn't really necessary, and has been removed. To ensure consistency with testing, `setuptools` has also been updated to the latest versions available.
    d051266bdc93bcbedd326d941db97c715699afb2

    Checks run (2 succeeded)

    flake8 passed.
    JSHint passed.
    david
    1. 
        
    2. Show all issues

      Interdiff seems empty?

    3.