docs(readme): fix 404 download links; drop unbuilt AppImage#82
Open
FourthWiz wants to merge 1 commit into
Open
docs(readme): fix 404 download links; drop unbuilt AppImage#82FourthWiz wants to merge 1 commit into
FourthWiz wants to merge 1 commit into
Conversation
The three Downloads links use unversioned asset names (releases/latest/download/NeuroSkill.dmg etc.), but every release asset is versioned (NeuroSkill_0.0.129_aarch64.dmg), so all three 404 permanently. Point to the releases page instead — name-agnostic, cannot rot. The Linux link also promised an AppImage that no workflow builds; the real Linux artifacts are .deb/.rpm/portable tar.gz.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The three Downloads links in the README all 404:
They can't ever resolve: every release asset is versioned (
NeuroSkill_0.0.129_aarch64.dmg,NeuroSkill_0.0.129_x64-setup.exe, …), so an unversionedreleases/latest/download/NeuroSkill.dmgURL has nothing to match. The Linux link also promises an AppImage, but no workflow builds one — the actual Linux artifacts on v0.0.129 are.deb,.rpm, and a portable.tar.gz.This points Downloads at the releases page instead — name-agnostic, so it can't rot when version numbers or asset names change — and names the real artifact types.
Left alone: the
brew tapline — the tap repo doesn't exist yet, tracked separately in #80.Docs-only change; changelog fragment included.