Skip to content

Conversation

@jqtrde
Copy link
Contributor

@jqtrde jqtrde commented Dec 24, 2025

This PR:

  • Updates action.yaml to use python -m build, which is necessary now that we've moved from setup.py to pyproject.
  • Bumps the Python version used in action.yaml to 3.11. (3.10 is the minimum we're supporting)
  • Adds a test extra so that we can still pip install -e .[test]

The release workflow needs an adjustment to work with pyproject.toml.
Otherwise, updating CI to use Python 3.11 to match the new Python
requirements, and adds a "test" extra to support the release workflow's
installation step.
@jqtrde jqtrde requested a review from a team as a code owner December 24, 2025 16:32
@jqtrde jqtrde requested review from bgdnatx and nicklvsa December 24, 2025 16:35
@jqtrde
Copy link
Contributor Author

jqtrde commented Dec 24, 2025

Tested this locally with:

# mimic the release environment
$ mise use [email protected]
$ rm -rf dist .build-venv
$ python -m venv .build-venv
$ source .build-venv/bin/activate.fish
$ python -m pip install --upgrade pip setuptools
$ python -m pip install build twine wheel

# wheel builds
$ python -m build --sdist --wheel
...
removing build/bdist.macosx-11.0-arm64/wheel
Successfully built mapbox_tilesets-2.0.0.tar.gz and mapbox_tilesets-2.0.0-py3-none-any.whl

# twine is happy
$ python -m twine check dist/*
Checking dist/mapbox_tilesets-2.0.0-py3-none-any.whl: PASSED
Checking dist/mapbox_tilesets-2.0.0.tar.gz: PASSED
$ deactivate

# install what was built
$ rm -rf venv
$ python -m venv venv
$ source venv/bin/activate.fish
$ pip install dist/*.tar.gz
Successfully built mapbox-tilesets

# confirm install works
$ tilesets --version
2.0.0

@jqtrde jqtrde merged commit c9f4db4 into master Dec 24, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants