Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixup version for maturin #837

Merged
merged 1 commit into from
Jan 1, 2025
Merged

fixup version for maturin #837

merged 1 commit into from
Jan 1, 2025

Conversation

altendky
Copy link
Contributor

@altendky altendky commented Jan 1, 2025

https://github.com/Chia-Network/chia_rs/actions/runs/12574101581/job/35047783400?pr=836

⚠️  Warning: `project.version` field is required in pyproject.toml unless it is present in the `project.dynamic` list
🍹 Building a mixed python/rust project
🔗 Found pyo3 bindings
🐍 Found CPython 3.10 at /home/ubuntu/actions-runner/_work/chia_rs/chia_rs/venv/bin/python
📡 Using build options features, bindings from pyproject.toml
⚠️  Warning: `project.version` field is required in pyproject.toml unless it is present in the `project.dynamic` list
💥 maturin failed
  Caused by: Cannot build without valid version information. You need to specify either `project.version` or `project.dynamic = ["version"]` in pyproject.toml.

https://github.com/PyO3/maturin/blob/main/Changelog.md#180
PyO3/maturin#2391
PyO3/maturin#2390

Copy link

Pull Request Test Coverage Report for Build 12574264424

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 83.972%

Totals Coverage Status
Change from base Build 12384648805: 0.0%
Covered Lines: 13349
Relevant Lines: 15897

💛 - Coveralls

@arvidn arvidn merged commit a404233 into main Jan 1, 2025
52 checks passed
@arvidn arvidn deleted the fixup_version_for_maturin branch January 1, 2025 22:54
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