← Back to issue list

Add electron-builder test for core20

View original Github issue

Metadata

Project
snapcraft
Number
#4182
Type
issue
State
open
Author
sergiusens
Labels
Status: Triaged Type: Enhancement
Created
Updated
Closed
—

Current evaluation

Requested core20 electron-builder spread coverage is obsolete: core20 support and related tests were removed in PRs #6019 and #6009, and core20 is now limited to the legacy Snapcraft 8.x track.

Suggested action: close stale

Reason: Repository history identifies PRs #6019 and #6009 as removing core20 support, while the current base reference limits core20 to the legacy Snapcraft 8.x track. No electron-builder spread test remains, so adding this core20-specific test to the current codebase is obsolete.

Evaluated by: mr-cal-bot

Impact: 5 Quick Win: 4 Complexity: 10 Confidence: 97 Actionability: 98

Issue body

### What needs to get done Add an electon-builder spread test using core20 as the base using the package.json syntax to select core20 (core18 is their default) ### Why it needs to get done With the removal of core18, we can no longer ensure electron-builder feature parity

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha
Impact: 5
Quick Win: 4
Complexity: 10
Confidence: 97
Actionability: 98
close stale Requested core20 electron-builder spread coverage is obsolete: core20 support and related tests were removed in PRs #6019 and #6009, and core20 is now limited to the legacy Snapcraft 8.x track.
qwen/qwen3.6-35b-a3b
Impact: 0
Quick Win: 0
Complexity: 10
Confidence: 95
Actionability: 100
close stale Request to add electron-builder spread test for core20. Core20 support has been completely dropped from the project in PR #6019 and #6009, making this request obsolete.
qwen/qwen3.8-27b
Impact: 35
Quick Win: 26
Staleness: 78
Complexity: 25
Confidence: 62
Support Request: 0
keep open Request to add an electron-builder spread test using core20 base (via package.json syntax) to preserve feature parity after core18 removal. Triaged by maintainer sergiusens; no implementation found in current codebase.
qwen/qwen3.6-35b-a3b
Staleness: 80
Complexity: 20
Confidence: 85
Support Request: 5
keep open Enhancement to add an electron-builder spread test using core20 as the base, required after core18 removal. Triaged by maintainer but inactive for over a year.
qwen3.6-35b-a3b-mtp-q6
Staleness: 85
Complexity: 35
Confidence: 75
Support Request: 10
keep open Enhancement to add an electron-builder spread test using core20 as the base, required after core18 removal to ensure feature parity. Currently triaged but inactive for over a year.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#4165 (confidence 98%)

    The core18 removal issue explicitly tracked the remaining electron-builder test cleanup.

  • Superseded By: snapcraft#6019 (confidence 96%)

    PR #6019 removed core20 support, eliminating the requested test target from the current project.

  • Superseded By: snapcraft#6009 (confidence 90%)

    PR #6009 was part of the removal of core20 support and related coverage.

Related issues

Issue Project State Summary Similarity
#4165 Remove core18 from Snapcraft snapcraft closed Core18 support was removed by merged PR #4177 in commits 40136ea9b, 77edc734b, and ed423b6a8; remaining electron-builder cleanup was tracked in #4182.
78%
#6009 test: remove core20 spread tests snapcraft merged Squash-merged PR #6009 as commit d0f1b4394, deleting core20 spread tests and migrating applicable ones to core22/core24 (+20 net tests). Unblocked core20 lifecycle removal for Snapcraft 9 (#5675).
78%
#5675 Remove core20 lifecycle snapcraft closed Merged PR #6019 via commit fa7952b4d, removing core20 build/lifecycle support and core20 command entries; core20 spread tests were removed separately in PR #6009.
76%
#3026 spread tests: add core20 and cleanup systems snapcraft merged Merged #3026 in commit d006134d4, adding core20 spread tests, simplifying setup to install only the Snap, and cleaning up test systems.
72%
#4177 core18: removal snapcraft merged Merged PR snapcore/snapcraft#4177 (May 2023) dropped core18 base support and its unit/spread tests, fixing #4165 via commits 40136ea9b, 77edc734b and ed423b6a8; core18 was EOL.
71%
#4895 fix(remotebuild): require core20 snaps to use the legacy remote builder snapcraft merged Merged in #4895 (d9c8781a1), requiring core20 remote builds to use the legacy builder because the new builder could not parse core20 snapcraft.yaml files; fixes #4886.
71%
#2761 Core20 base snapcraft merged Core20 base support was merged via #3579, moving Snapcraft to core20 in commit 18a974e01 (merge ed2baf8bd).
71%
#2009520 core18/20: inconsistent architecture parsing snapcraft (launchpad) closed Superseded by snapcore/snapcraft#4340 and #4341. Core22+ overlapping build-on handling merged in #3821 (4876b53); core20 support was declined.
71%