← Back to issue list

Updated the documentation about the icon.

View original Github issue

Metadata

Project
snapcraft
Number
#542
Type
pull request
State
merged
Author
come-maiz
Labels
—
Created
Updated
Closed

Current evaluation

Docs-only PR merged as a9f2a7d7a: tutorial no longer tells users to set `icon` in snapcraft.yaml (deprecated) but to place icon.svg/icon.png under setup/gui. Closes LP bug #1578231.

Resolution: closed resolved

Reason: The PR itself was merged (commit a9f2a7d7a, 'Updated the documentation about the icon. (#542)') after a +1 from sergiusens, updating the tutorial to describe setup/gui icons instead of the deprecated snapcraft.yaml `icon` key. This closed the linked Launchpad bug LP: #1578231.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Thanks Jamie for the wording. LP: #1578231

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Docs-only PR merged as a9f2a7d7a: tutorial no longer tells users to set `icon` in snapcraft.yaml (deprecated) but to place icon.svg/icon.png under setup/gui. Closes LP bug #1578231.
stealth/space-bunny-alpha — closed resolved Merged in snapcraft PR #542 (a9f2a7d7a), replacing deprecated snapcraft.yaml icon guidance with setup/gui icons in the tutorial, resolving LP #1578231.
qwen/qwen3.6-35b-a3b — — Documentation update regarding icon usage merged. Changes added 19 lines and removed 2 across one file. Approved by sergiusens and merged without CI checks or reviewer conflicts.
qwen3.6-35b-a3b-mtp-q6 — — Merged after incorporating feedback on icon documentation wording. Approved by @sergiusens and merged by maintainer come-maiz.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft (launchpad)#1578231 (confidence 98%)

    LP bug #1578231 'Your first snap: Use of icon in snapcraft.yaml is deprecated' is the upstream bug this PR resolves

  • Related To: snapcraft#914 (confidence 70%)

    Later commit 4a4435cbe 'meta: icon is now dedeprecated (#914)' formalised the deprecation this PR documented around

Related issues

Issue Project State Summary Similarity
#914 meta: icon is now dedeprecated snapcraft merged PR #914 merged as commit 4a4435cbe, re-deprecating the icon deprecation: the icon entry in snapcraft.yaml is again the preferred method, overriding any icon in the setup directory (LP: #1610268).
73%
#1578231 Your first snap: Use of icon in snapcraft.yaml is deprecated snapcraft (launchpad) closed Docs fixed in canonical/snapcraft#542 (commit a9f2a7d7a): the 'Your first snap' tutorial now tells users to place icon.png/icon.svg under setup/gui instead of the deprecated icon key. Shipped in snapcraft 2.10+16.10 and 2.10.1.
73%
#409 Explain the new way to provide snap icons. snapcraft merged Merged guidance clarifying the supported way to provide snap icons in snapcraft.yaml, resolving LP #1561327 (merge 4073f89fc).
70%