build: remove unnecessary constraints
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Removed unnecessary dependency constraints from pyproject.toml after a maintainer confirmed PyYAML 6.0.1 natively supports Cython 3, rendering the pinned constraints redundant. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #811 build(deps): drop unnecessary pydantic-yaml | craft-parts | merged | Merged to remove the pydantic-yaml dependency, as Pydantic 2 uses model_dump() with PyYAML directly. Approved by two reviewers and passed CI checks. | |
| #2829 fix(test): add ops>=2.0.0 to uv constraint-dependencies | charmcraft | merged | Added ops>=2.0.0 to uv constraint-dependencies in pyproject.toml and the uv_project test fixture. Incorporated reviewer feedback to constrain pyyaml>=5 and fix formatting. Approved by three reviewers and merged. |