Repository navigation
feat: publish mailtrap-local to the Snap Store - #53
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (4)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe change adds Snap packaging for amd64 and arm64, CI smoke tests, Snap Store publication, release asset verification, and README installation instructions. ChangesSnap distribution
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to The Snap packaging and release automation change has no identified actionable merge-blocking risk at the current head; it is merge-ready after normal checks and review. Sequence Diagram(s)sequenceDiagram
participant GitHubActions
participant GoReleaser
participant Snapcraft
participant SnapStore
participant GitHubRelease
GitHubActions->>GoReleaser: run release with Snapcraft credentials
GoReleaser->>Snapcraft: publish stable amd64 and arm64 packages
Snapcraft->>SnapStore: upload Snap packages
GoReleaser->>GitHubRelease: upload Snap assets
GitHubActions->>GitHubRelease: verify amd64 and arm64 Snap assets
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.) Full details: Description checkExplanation The description explains the motivation, key changes, testing steps, Snap persistence behavior, and release prerequisites. It omits the template's separate Changes and Images and GIFs sections, but the relevant change information is present and the missing images section is non-critical. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What & why
Ubuntu desktop is a large share of Linux installers, and we had no Snap channel. Goreleaser now packs linux amd64/arm64 snaps and uploads them to the Snap Store on each
v*tag (stable only), same pipeline as Docker Hub.core22snap; DB + secret key under$SNAP_USER_COMMON(~/snap/mailtrap-local/common/).snap/gui/icon.png).snapjob: snapshot pack +snap install --dangerous+ SMTP smoke (no store).Release environment still needs
SNAPCRAFT_STORE_CREDENTIALS. First store upload waits on Canonical review.How to test
v*tag (and Canonical review),sudo snap install mailtrap-localon Ubuntu.mailtrap-local --version; UI at http://127.0.0.1:3550; SMTP at 127.0.0.1:3535.~/snap/mailtrap-local/common/across a snap refresh.Summary by CodeRabbit
New Features
Documentation
Tests