← Back to issue list

chore: remove unused docs requirements file

View original Github issue

Metadata

Project
craft-parts
Number
#1357
Type
pull request
State
merged
Author
bepri
Labels
—
Created
Updated
Closed

Current evaluation

Merged #1357 as commit b472871d, deleting the unused 349-line docs/requirements.txt to stop needless Renovate PR noise; all CI checks passed.

Resolution: closed resolved

Reason: PR canonical/craft-parts#1357 was merged as commit b472871d, removing the obsolete requirements file after approval and with all CI checks passing.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint && make test`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? --- This requirements file isn't used by any of our workflows and just [generates noise from renovate PRs](https://github.com/canonical/craft-parts/pull/1352). It was decided some time ago to [just remove it in Snapcraft](https://github.com/canonical/snapcraft/pull/5598), so I'm doing the same here.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #1357 as commit b472871d, deleting the unused 349-line docs/requirements.txt to stop needless Renovate PR noise; all CI checks passed.
stealth/space-bunny-alpha — closed resolved Merged craft-parts#1357 (b472871d), deleting an unused 349-line docs requirements file to eliminate unnecessary Renovate dependency-update noise. All CI checks passed.
qwen/qwen3.6-35b-a3b — — Merged removal of an unused docs requirements file to eliminate Renovate PR noise. Approved by two reviewers with all CI checks passing, following a prior cleanup in the Snapcraft project.
qwen3.6-35b-a3b-mtp-q6 — — Merged a maintainer cleanup removing an unused documentation requirements file to eliminate Renovate PR noise and align with a prior Snapcraft repository cleanup.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#5598 (confidence 100%)

    A previously merged Snapcraft PR removed its equivalent unused Sphinx requirements file.

  • Caused By: canonical/craft-parts#1352 (confidence 95%)

    The obsolete requirements file prompted noisy Renovate dependency-update PRs.

Related issues

Issue Project State Summary Similarity
#1776 chore: remove unused code charmcraft merged Merged PR #1776 as commit 027696be, deleting 17,571 lines of unused code across 36 files after maintainer review and approval.
78%
#165 chore: deprecate requirements*.txt craft-parts merged Merged PR #165 as 7625d623 (dc734c84), removing requirements*.txt and leaving package and version metadata in setup.py.
77%
#2823 chore: configure renovate to ignore requirements files under tests/ charmcraft merged PR #2823 merged, adding `tests/**` to ignorePaths in .github/renovate.json5 so Renovate stops raising updates for test-fixture requirements files.
76%
#734 chore(deps): update documentation dependencies craft-parts closed Renovate PR #734 was closed unmerged after five days of inactivity; its dependency versions appeared in 84e96162 and were later carried forward by 67623b5d.
74%
#2444 chore: clean up unused/unnecessary files charmcraft merged Merged maintenance PR #2444 (commit 6b94f660) removing 286 lines across 10 unused/unnecessary files after two approvals; macOS smoke-test jobs failed but did not block the merge.
74%
#2224 chore: remove tox and freeze-requirements files charmcraft merged Merged #2224 in commit 484d6ed0, removing obsolete tox and frozen-requirements files (212 lines).
74%
#6079 chore: delete unused file snapcraft merged Merged PR #6079 as commit 75686f3da, deleting a three-line file unused since Snapcraft moved to a pyproject-based setup.
74%
#156 chore: make renovate update pre-commit-config starbase merged Merged PR #156 as commit 46e9d22, updating Renovate so dependencies in the pre-commit configuration are maintained automatically.
73%
#547 chore(deps): update documentation dependencies craft-parts merged Renovate docs-dependency bump PR merged to main as a9512830 after lengau manually downgraded some packages once the Read the Docs build passed via #556; the failing lint was unrelated (#562).
72%
#1830 chore: remove unused file `bases.py` charmcraft merged Merged PR #1830 removed the unused 169-line bases.py module; commit 9a35c1ec records the cleanup.
72%