← Back to issue list

requirements: unpin pyyaml and update dependencies

View original Github issue

Metadata

Project
snapcraft
Number
#3713
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged following reviewer approval and passing CI. Unpinned mccabe and PyYAML to enable libyaml bindings for Python 3.10, updating four dependency files.

Suggested action: —

No scores available.

Issue body

Unpin mccabe and PyYAML, allow libyaml binding with python 3.10. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged following reviewer approval and passing CI. Unpinned mccabe and PyYAML to enable libyaml bindings for Python 3.10, updating four dependency files.
qwen3.6-35b-a3b-mtp-q6 — — Merged changes that unpinned mccabe and PyYAML, enabling libyaml bindings for Python 3.10. Integrated by maintainer cmatsuoka.
qwen3.6-35b-a3b-mtp-q6 — — Merged change unpinned mccabe and PyYAML dependencies, enabling libyaml bindings for Python 3.10. Integrated by maintainer cmatsuoka.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3490 build(deps): bump pyyaml from 5.3 to 5.4 snapcraft merged Merged Dependabot update bumping pyyaml from 5.3 to 5.4. Required test updates and a rebase were completed before approval and successful CI merge.
71%
#2923 requirements: Update PyYAML requirement to 5.3 snapcraft merged Merged update to PyYAML 5.3 to resolve Windows wheel availability for Python 3.8+. Reviewer confirmed standard pip installation works, so custom wheel references were removed prior to merge.
70%
#332 build(deps): drop pyyaml dependency craft-cli merged Merged to drop the unused pyyaml dependency ahead of a major release. Approved by two reviewers, passed CI checks, and modified a single file.
70%