← Back to issue list

requirements: update dependencies

View original Github issue

Metadata

Project
snapcraft
Number
#4011
Type
pull request
State
merged
Author
tigarmo
Labels
—
Created
Updated
Closed

Current evaluation

Merged routine pinned-requirements refresh by tigarmo into snapcraft main as commit 42b868319 (5 files, +84/-74), approved by 2 reviewers; setuptools megabump deliberately excluded for a separate ruff PR.

Resolution: closed resolved

Reason: The PR itself was merged: commit 42b868319 "requirements: update dependencies (#4011)" landed in snapcraft main after approval by 2 reviewers, and Codecov reported no coverage change. The only open review thread was a deliberate choice to defer the setuptools 49.6.0 -> 65.5.1 megabump.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint`? - [ ] Have you successfully run `pytest tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged routine pinned-requirements refresh by tigarmo into snapcraft main as commit 42b868319 (5 files, +84/-74), approved by 2 reviewers; setuptools megabump deliberately excluded for a separate ruff PR.
stealth/space-bunny-alpha — closed resolved Merged dependency updates in canonical/snapcraft#4011 (commit 42b868319), refreshing pinned requirements following two maintainer approvals.
stealth/space-bunny-alpha — closed resolved Merged dependency updates via #4011 (commit 42b868319), keeping them separate from the follow-up Ruff and freeze-requirements work.
qwen/qwen3.6-35b-a3b — — Dependencies updated and merged following two approvals and passing CI. Changes were isolated from a separate ruff configuration update to prevent commit pollution.
qwen3.6-35b-a3b-mtp-q6 — — Merged dependency updates. Routine requirement changes were isolated from a separate ruff configuration PR to avoid large setuptools version jumps. Code coverage remained unchanged at 91.61% with 100% diff coverage.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#4007 (confidence 80%)

    setuptools 49.6.0->65.5.1 bump discussed in review as a separate, deliberately excluded change; it closed unmerged since freeze-requirements.sh would overwrite it.

  • Related To: canonical/craft-providers@ab325f27215ae669e6a9f41c507fa79a28f33b19 (confidence 55%)

    Cited by mr-cal as the equivalent freeze-requirements change in craft-providers (codespell/similar pins).

Related issues

Issue Project State Summary Similarity
#3948 requirements: update snapcraft merged Merged routine dependency refresh into main as 2a0ce7cfe (branch commit 2a122048e), bumping pinned requirements across 11 files (+78/-75) after one approving review.
79%
#6386 build(deps): update dependency setuptools to v84 (main) snapcraft merged Renovate PR #6386 bumped setuptools from ~=83.0.0 to ~=84.0.0 in Snapcraft's main branch; approved with all core CI green and squash-merged as commit afde69391 (+5/-5 lines, 2 files).
76%
#874 build(deps): update dependencies rockcraft merged Merged routine dependency-bump PR by maintainer tigarmo into main as commit 452a71dc (+1806/-1430 across 5 files), with 2 approvals and all lint, fast/slow, min-deps, schema and security checks green.
72%
#6087 build(deps): update dependency setuptools to v82 (main) snapcraft closed Renovate PR to bump setuptools to v82 was closed unmerged by @mr-cal ("Waiting to drop snapcraft legacy"); after snapcraft_legacy removal, PR #6386 (commit afde69391) landed the bump to setuptools~=84.0.0.
70%