← Back to issue list

misc: remove snapd "submodule"

View original Github issue

Metadata

Project
snapcraft
Number
#1051
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #1051 as commit f62ac0ce1, removing the obsolete snapd testing-tools submodule reference in a one-line cleanup.

Resolution: closed resolved

Reason: The approved PR was merged into master as commit f62ac0ce1. It deleted the snapd testing-tools entry from .gitmodules without changing code coverage.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #1051 as commit f62ac0ce1, removing the obsolete snapd testing-tools submodule reference in a one-line cleanup.
stealth/space-bunny-alpha — closed resolved Merged snapcraft#1051 as commit f62ac0ce1, deleting the snapd "submodule" reference in an approved one-line cleanup that left coverage unchanged.
qwen/qwen3.6-35b-a3b — — Merged changes removing the snapd submodule reference. The update deleted one line in a single file, received one approval, and integrated into master without affecting code coverage.
qwen3.6-35b-a3b-mtp-q6 — — Merged into master, removing the snapd submodule. Code coverage remained unchanged at 96.35%. The maintainer integrated the change without affecting test metrics.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#106 tools: update snapd-testing-tools to latest rockcraft merged Merged PR #106 as 57a6e25d, re-applying the snapd-testing-tools submodule update after an accidental revert and resolving a codespell-detected typo.
72%
#5689 fix: remove unnecessary snapd_snap flag (#5316) snapcraft merged Merged #5689 as 8f02d3d48, removing the unnecessary snapd_snap flag and fixing #5316; the change was also backported in #5694.
72%
#2421 tests: remove obsolete snap and external tests snapcraft merged Merged PR #2421 in commit 707395546, deleting 77 obsolete snap/external-test files and 2,959 lines despite a negligible coverage decrease.
71%