Merge lp:~gary-wzl77/ubuntu-weather-app/snap_fix_1552100 into lp:ubuntu-weather-app
Proposed by
Gary.Wang
Status: | Merged |
---|---|
Approved by: | Andrew Hayzen |
Approved revision: | 225 |
Merged at revision: | 252 |
Proposed branch: | lp:~gary-wzl77/ubuntu-weather-app/snap_fix_1552100 |
Merge into: | lp:ubuntu-weather-app |
Prerequisite: | lp:~gary-wzl77/ubuntu-weather-app/fix_1518888 |
Diff against target: |
209 lines (+194/-0) 2 files modified
setup/gui/ubuntu-weather-app.desktop (+161/-0) snapcraft.yaml (+33/-0) |
To merge this branch: | bzr merge lp:~gary-wzl77/ubuntu-weather-app/snap_fix_1552100 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jenkins Bot | continuous-integration | Approve | |
Andrew Hayzen | Approve | ||
Review via email: mp+311681@code.launchpad.net |
Commit message
create snap package by using ubuntu-app-platform plugin.
Description of the change
create snap package by using ubuntu-app-platform plugin.
To post a comment you must log in.
Awesome! This looks like a good start, there is one inline comment about the plugs. But I think the main thing missing is there is no setup/gui folder with an icon and .desktop file (see the packaging icon and desktop file docs here http:// snapcraft. io/docs/ build-snaps/ metadata# package- icon). You should also be able to modify the icon and .desktop file from the existing click ones.