← Back to issue list

build: remove unnecessary constraints

View original Github issue

Metadata

Project
snapcraft
Number
#5488
Type
pull request
State
merged
Author
bepri
Labels
—
Created
Updated
Closed

Current evaluation

Merged #5488 in commit 3d6c39f0d, removing three obsolete build dependency constraints after PyYAML 6.0.1 gained proper Cython 3 support.

Resolution: closed resolved

Reason: PR #5488 was merged as commit 3d6c39f0d. All CI checks passed, and the change removed the unnecessary constraints.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint`? - [ ] Have you successfully run `make test`? --- Constraint dependencies can be specified in the pyproject.toml, which declutters the project at the top-level.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #5488 in commit 3d6c39f0d, removing three obsolete build dependency constraints after PyYAML 6.0.1 gained proper Cython 3 support.
stealth/space-bunny-alpha — closed resolved Merged #5488 (commit 3d6c39f0d), removing three obsolete dependency constraints after PyYAML 6.0.1 added Cython 3 support.
qwen/qwen3.6-35b-a3b — — Merged PR removes unnecessary dependency constraints from pyproject.toml. Updated after a reviewer noted PyYAML 6.0.1 natively supports Cython 3. Approved by two reviewers with all CI checks passing.
qwen3.6-35b-a3b-mtp-q6 — — Merged after removing unnecessary constraint dependencies. PyYAML 6.0.1 now natively supports Cython 3, making them obsolete. Updated and merged to declutter the build configuration.

Update history

No update history recorded yet.

Related work

  • Likely Fixed By: pyyaml#808 (confidence 99%)

    PyYAML 6.0.1 added proper Cython 3 support, making the removed constraints obsolete.

Related issues

Issue Project State Summary Similarity
#811 build(deps): drop unnecessary pydantic-yaml craft-parts merged Merged #811 in commit 1debce16, removing pydantic-yaml because Pydantic 2 models are serialized with model_dump() and PyYAML directly.
71%