Skip to content

fix: usability improvements across CLI, progress display, and error h…#3

Open
jodevsa wants to merge 1 commit into
mainfrom
fixes_1
Open

fix: usability improvements across CLI, progress display, and error h…#3
jodevsa wants to merge 1 commit into
mainfrom
fixes_1

Conversation

@jodevsa

@jodevsa jodevsa commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

…andling

  • Fix misleading progress display (metric count shown as duration)
  • Add log message before silent 5s InfluxDB wait
  • Fix buildGrafanaURL to return (string, error) idiomatically
  • Fix silent error swallowing in stream.go for file open failures
  • Fix wrong --metrics-include-file flag description in config.go
  • Add --metrics-include-file option and input validation to run.sh
  • Replace hardcoded container name with docker-compose logs -f in run.sh

…andling

- Fix misleading progress display (metric count shown as duration)
- Add log message before silent 5s InfluxDB wait
- Fix buildGrafanaURL to return (string, error) idiomatically
- Fix silent error swallowing in stream.go for file open failures
- Fix wrong --metrics-include-file flag description in config.go
- Add --metrics-include-file option and input validation to run.sh
- Replace hardcoded container name with docker-compose logs -f in run.sh

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant