Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 15 additions & 6 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,19 @@

## Checklist

- [ ] Code follows the project conventions (see `.ai/skills/new-command.md`)
- [ ] `go build ./...` passes
- [ ] `go test ./...` passes
- [ ] `--debug` output included for API-calling commands
- [ ] Destructive actions have confirmation prompts
- [ ] I have performed a self-review of my own code
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally (`make test`)
- [ ] Pre-commit checks pass (`make pre-commit`)
- [ ] My changes generate no new warnings or errors

> **Note:** The CHANGELOG is auto-generated from conventional commit messages at release time. Do not edit CHANGELOG.md manually.
## Related Issues

<!-- Link any related issues: Closes #123, Related to #456 -->

## Additional Context

<!-- Screenshots, benchmarks, or other context (optional) -->

> **Note:** Use [conventional commit](https://www.conventionalcommits.org/) messages (e.g. `feat:`, `fix:`, `chore:`).
> The CHANGELOG is auto-generated from conventional commit messages at release time.
7 changes: 7 additions & 0 deletions internal/verda-cli/cmd/update/update.go
Original file line number Diff line number Diff line change
Expand Up @@ -139,6 +139,13 @@ func runUpdate(ctx context.Context, f cmdutil.Factory, ioStreams cmdutil.IOStrea
}

if err := replaceBinary(exe, binary); err != nil {
if errors.Is(err, os.ErrPermission) {
hint := "sudo verda update"
if runtime.GOOS == osWindows {
hint = "running the command in an elevated (Administrator) terminal"
}
return fmt.Errorf("permission denied writing to %s\n\nTry: %s", filepath.Dir(exe), hint)
}
return fmt.Errorf("replacing binary: %w", err)
}

Expand Down
Loading