← Back to issue list

snap: remove completer entry

View original Github issue

Metadata

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

Current evaluation

Merged PR #1409 as db235805a, removing the unsupported completer entry so snapcraft.yaml remained compatible with 2.32 pending 2.33 rollout on Xenial.

Resolution: closed resolved

Reason: PR #1409 was approved and merged as commit db235805a. Its metadata change removed the completer entry, resolving the Snapcraft 2.32 YAML compatibility problem.

Evaluated by: mr-cal-bot

No scores available.

Issue body

We should rever tthis as soon as 2.33 hits xenial. There's a catch-22 since 2.32 does not accept a completer entry in the yaml. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #1409 as db235805a, removing the unsupported completer entry so snapcraft.yaml remained compatible with 2.32 pending 2.33 rollout on Xenial.
stealth/space-bunny-alpha — closed resolved Merged PR #1409 (db235805a) removed the completer entry from snapcraft.yaml, keeping compatibility with 2.32 pending the 2.33 rollout.
qwen/qwen3.6-35b-a3b — — Merged a one-line change removing a completer entry from the snap YAML. This served as a temporary workaround for snapd 2.32 compatibility, with plans to revert once version 2.33 reaches Xenial. Approved by one reviewer.
qwen3.6-35b-a3b-mtp-q6 — — Merged removal of the completer entry from the snap YAML to resolve a compatibility catch-22 with snapd 2.32 on Xenial. This change acts as a temporary workaround pending snapd 2.33 release.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#1956 (confidence 98%)

    Later merged PR #1956, commit 32fbaf3d4, re-enabled shell completion after the compatibility constraint was resolved.

Related issues

Issue Project State Summary Similarity
#1751391 snapcraft's snap's snap.yaml doesn't plug in the completer snapcraft (launchpad) closed Snap completion was fixed by PR #1956 (commit 32fbaf3d4), adding completer: completion.sh to Snapcraft's snap app definition.
74%
#1956 snap: actually plug the completer in. snapcraft merged PR #1956 merged as commit 32fbaf3d4, wiring the bundled completion.sh into snap metadata so Snapcraft exposes tab completion.
71%
#1373 snapcraft.yaml: add support for reload-command and completer directives snapcraft merged Merged snapcraft#1373 in commit 48a347edf, adding snapcraft.yaml support for the reload-command and completer directives.
70%
#1590767 Support snap installed completion scripts snapcraft (launchpad) closed Resolved by snapd 2.28 automatic completion integration and snapcraft completer wiring in merged PR #1956 (commit 32fbaf3d4).
70%