Skip to content

Sdist and wheel#235

Merged
fohrloop merged 9 commits into
devfrom
sdist-and-wheel
Apr 1, 2024
Merged

Sdist and wheel#235
fohrloop merged 9 commits into
devfrom
sdist-and-wheel

Conversation

@fohrloop
Copy link
Copy Markdown
Member

@fohrloop fohrloop commented Apr 1, 2024

  1. Create sdists with also docs + tests included and wheels with just the wakepy package. Sdists will contain all the repo contents that is not gitignored (using setuptools-scm). This way the sdist will always have all contents required to run tests, create documentation and create the wakepy wheel.

  2. Version string & tuple
    Take the version from git tags using setuptools-scm. The setuptools-scm is also responsible of creating wakepy._version
    module from which wakepy.__version__ and wakepy.version_tuple are read. Previously, the version string was defined in wakepy.__init__.py as variable (__version__).

  3. Improve the build process: Now all errors are printed to stderr (tox colors these with red) and there is possibility to filter errors.

Transition from flit to setuptools. Just for setuptools-scm.

fohrloop added 7 commits April 1, 2024 18:57
Just for using setuptools-scm, which makes it easier to keep the files
included in the sdist in sync.
…l does not contain a tool.setuptools_scm section'
* print to stdout and stderr everything
1) no longer needed since using src layout
2) this produced empty wheels
@fohrloop
Copy link
Copy Markdown
Member Author

fohrloop commented Apr 1, 2024

The build and tests work well when running locally. In GitHub Actions the python 3.7 fails to build. That's a pipeline issue which I'm going to address in a separate PR. The wheel should anyway be built once and tested with multiple platforms and versions of python. Merging this one.

@fohrloop fohrloop merged commit eb2a8b5 into dev Apr 1, 2024
@fohrloop fohrloop deleted the sdist-and-wheel branch April 3, 2024 21:45
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.

1 participant