fix(release-notes): expand Downloads section with AppImage, .deb, and Tauri mention - #1454
TimeToBuildBob wants to merge 1 commit into
Conversation
… Tauri mention Add .deb and AppImage links for Linux, an Apple Silicon .dmg link for macOS, and a note pointing to the Tauri-based alternate build (which additionally supports Wayland on Linux). Addresses the release-notes part of ActivityWatch#1453 (the activitywatch.net/downloads page still needs the same Tauri mention, tracked separately in that issue). Git-Session-Id: 332a6b70-940a-5f28-b303-f608651f1313
|
|
CI failure investigated ( Root cause is a macOS runner environment issue inside I don't have admin rights on this repo to trigger |
🤖 AI code reviewThe PR updates the Downloads section of the auto-generated release notes in scripts/build_changelog.py. It adds an Apple Silicon macOS .dmg link, adds .deb and AppImage links for Linux, and appends a note about an alternate Tauri-based build with a pointer to the aw-tauri directory and a mention of Wayland support on Linux. Not safe to merge — 1 P1 openConfidence 3/5 1 finding · ❌ 1 P1❌ P1 high — The new AppImage link uses the filename pattern How this was verified: Compared the AppImage URL pattern against the .deb and .zip URLs in the same line and against the macOS/Windows patterns; all other non-Tauri assets include the {tag} segment. Checked the PR description which acknowledges naming inconsistency only for Tauri assets, not for the standard AppImage. 1 advisory finding (summary-only, not scored)These P2 guard, heuristic, trade-off, or documentation claims are retained for judgment without opening review threads.
This is a How this was verified: static preflight: fix-commit + touched-files scan (rule 7) Files changed (1) — the diff as I read it
Reviewed Maintainer commands
|
| - [**macOS**](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-macos-x86_64.dmg) (.dmg) | ||
| - [**Linux**](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-linux-x86_64.zip) (.zip) | ||
| - [**macOS**](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-macos-x86_64.dmg) (.dmg, Intel) / [Apple Silicon](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-macos-arm64.dmg) | ||
| - [**Linux**](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-linux-x86_64.zip) (.zip) / [.deb](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-linux-x86_64.deb) / [AppImage](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-linux-x86_64.AppImage) |
There was a problem hiding this comment.
❌ P1 — The new AppImage link uses the filename pattern activitywatch-linux-x86_64.AppImage, but the actual release assets for ActivityWatch use a versioned pattern like activitywatch-{tag}-linux-x86_64.AppImage (consistent with the other links in the same section, e.g. the .deb and .zip). The PR description itself notes that Tauri asset naming is inconsistent, but the AppImage link is for the non-Tauri build, which follows the versioned convention. As written, the generated release notes will point to a URL that does not exist for any release, producing a 404 for users clicking the AppImage link. The fix is to include the {tag} segment in the filename, matching the .deb and .zip links.
|
Superseded by #1455 which has a more complete implementation (handles Tauri AppImage naming, extracts base URL, adds proper sections). Closing this in favor of #1455. Note: both PRs have a pre-existing CI failure in 'Build Qt artifacts (macos-latest)' — the error is a poetry/pip venv conflict (aw_core dist-info missing during aw-watcher-afk downgrade) that also appears on master. It is unrelated to this PR's changes (which only touch scripts/build_changelog.py). |
Summary
The auto-generated release notes' Downloads section (
scripts/build_changelog.py) only listed the plain Windows.exe, macOS.dmg, and Linux.zipassets, even though releases now also ship a.deb, an AppImage, an Apple Silicon.dmg, and a full Tauri-based alternate build for all three platforms (see the assets on e.g. v0.14.0b7).This adds:
.deband AppImage links for Linux.dmglink for macOSScope
This fixes the release-notes part of #1453. The issue also mentions that https://activitywatch.net/downloads/ is missing the Tauri mention — I didn't find the source repo for that page from this checkout, so that part is left for a follow-up (or a pointer to the right repo).
I deliberately didn't hardcode exact Tauri asset filenames/links: the naming isn't fully consistent across platforms/formats in the current release assets (some use a
vprefix, some don't), so a prose pointer to theactivitywatch-tauri-*assets is safer than a link that could go stale or 404.Test plan
python3 -m py_compile scripts/build_changelog.py