Skip to content

Conversation

@petrasvestartas
Copy link
Collaborator

@petrasvestartas petrasvestartas commented Jun 4, 2025

Pull request to
a) test the build action
b) install tessagon on the previously failing cibuildwheels test

@petrasvestartas
Copy link
Collaborator Author

petrasvestartas commented Jun 4, 2025

For cibuildwheels, requirements must be specified as follows in pyproject.toml:

Build:

requires = ["scikit-build-core >=0.10", "nanobind >=1.3.2"]

Test:

test-requires = ["numpy", "compas", "pytest", "build", "tessagon"]

Since tessagon is only used in testing, it is enough to specify it in test-requires list.

This building process is different from the local build, that is why requirements.txt were not working.
In total we are building twice:

  1. one time on a local os on one single python version
  2. second time on cibuildwheels for all python versions

Copy link
Member

@tomvanmele tomvanmele left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jeeej

@petrasvestartas petrasvestartas merged commit e7ed598 into main Jun 11, 2025
9 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.

3 participants