-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add absolute deadline column to today and tomorrow commands #198
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 Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Comment |
Co-authored-by: narthur <[email protected]>
Co-authored-by: narthur <[email protected]>
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-198-latest buzz-pr-198# Run the pre-release
buzz-pr-198# Uninstall the pre-release
bin remove buzz-pr-198Direct 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 enhances the buzz today and buzz tomorrow commands by adding an absolute deadline column alongside the existing relative deadline, making it easier for users to see both how much time remains and the exact date/time when goals are due.
Key changes:
- Added
FormatAbsoluteDeadline()function with context-aware formatting (time-only for today, "tomorrow" prefix for tomorrow, date+time for future dates) - Updated
handleFilteredCommand()to calculate and display a fourth column with proper alignment - Added comprehensive test coverage for the new formatting function
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| beeminder.go | Implements FormatAbsoluteDeadline() and FormatAbsoluteDeadlineAt() functions with timezone-aware date/time formatting logic |
| beeminder_test.go | Adds TestFormatAbsoluteDeadline with test cases covering today, tomorrow, and future date formatting scenarios |
| main.go | Updates handleFilteredCommand() to calculate column width for relative deadlines and add absolute deadline as fourth column |
| README.md | Updates documentation and examples to show the new absolute deadline column in command outputs |
|
@copilot apply changes based on the comments in this thread |
Co-authored-by: narthur <[email protected]>
Co-authored-by: narthur <[email protected]>
Plan: Address PR Review Comments
Summary
All review comments have been addressed:
All tests pass (17 new test cases added).
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.