Skip to content

Point the install scripts' source fallback at the src/ module - #45

Merged
imanimanyara merged 1 commit into
mainfrom
fix/install-fallback-src-path
Nov 8, 2026
Merged

imanimanyara merged 1 commit into
mainfrom
fix/install-fallback-src-path

Conversation

@imanimanyara

Copy link
Copy Markdown
Member

Summary

  • src/scripts/install.sh and install.ps1 fall back to go install when the release download fails. They still named github.com/simtabi/vigil/cmd/vigil@latest, the path from before the module moved into src/, which resolves to the frozen 0.7.0.
  • Both now name github.com/simtabi/vigil/src/cmd/vigil@latest. No src/ tag exists yet, so this builds the default branch.
  • CHANGELOG [Unreleased] gains a Fixed entry.

Testing

  • bash -n src/scripts/install.sh
  • go -C src test ./...: all packages pass except internal/netcheck, which fails locally only because the sandbox blocks httptest port binding. CI runs it.
  • make build writes build/macos/vigil, and TestBuildLayout passes.

The fallback still named github.com/simtabi/vigil/cmd/vigil, the path from
before the module moved into src/, so it resolved to the frozen 0.7.0.
Copilot AI balanced review requested due to automatic review settings November 8, 2026 09:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@imanimanyara
imanimanyara merged commit 67c9f2a into main Nov 8, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants