-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add minutes support to buzz due command #205
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
Co-authored-by: narthur <[email protected]>
🚀 Pre-release Build ReadyTest builds are ready! Install directly using the # Install the pre-release
bin install https://github.com/PinePeakDigital/buzz/releases/tag/pr-205-latest buzz-pr-205# Run the pre-release
buzz-pr-205# Uninstall the pre-release
bin remove buzz-pr-205Direct Download LinksOr download binaries directly from the pre-release page:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds minute-granularity support to the buzz due command, enabling queries like buzz due 10m to find goals due within the next 10 minutes. The implementation adds m/M as a duration unit alongside existing hours, days, and weeks.
- Added minutes case to
ParseDurationfunction to handlem/Msuffix - Updated comprehensive test coverage including edge cases (fractional minutes, uppercase, negative, zero)
- Updated all user-facing documentation (help text, error messages, README) to include minutes in examples and supported units
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| beeminder.go | Added minutes case to ParseDuration switch statement and updated function documentation |
| beeminder_test.go | Added 8 new test cases for minutes functionality (1m, 10m, 30m, 0.5m, uppercase M, negative, zero) and converted invalid "5m" test to use "5s" |
| main.go | Updated help text and error messages to include minutes in examples and supported units list |
| README.md | Added minutes to documentation with example usage and supported units list |
Adds
m/Mas a duration unit for thebuzz duecommand, enabling minute-granularity queries likebuzz due 10m.Changes
m/Msuffix totime.MinuteExample
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
Note
Adds minute-level durations to
buzz due, updating parsing, help text, and docs.ParseDurationto supportm/M(including fractional minutes) alongside hours/days/weeksWritten by Cursor Bugbot for commit 04596b8. This will update automatically on new commits. Configure here.