feat: add --minutes flag to timesheet command (#79) #82

Merged

kfickel merged 3 commits from 79_timesheet-use-minutes-flag into main 2026-04-07 08:13:21 +02:00

Conversation 0 Commits 3 Files changed 6 +81 -27

kfickel commented 2026-04-07 07:57:59 +02:00

Owner

Copy link

Summary

• Adds -m/--minutes flag to streamd timesheet command
• When set, time is displayed as HH:MM (e.g., 8:30) instead of decimal hours (e.g., 8.5h)
• Applies everywhere: per-day columns, monthly totals, cumulative balance, and outside-period warnings
• Negative diffs display with a leading minus sign (e.g., -1:30)

Changes

• src/cli/args.rs: Converted Timesheet from unit variant to struct variant with --minutes / -m bool flag
• src/main.rs: Destructures the new flag and passes it to timesheet::run()
• src/cli/commands/timesheet.rs: Updated format_hours() and format_diff() to accept use_minutes: bool; propagated flag through all print helpers; added 8 unit tests

Test plan

• cargo test — all 164 tests pass
• cargo clippy — no warnings
• streamd timesheet — unchanged decimal output
• streamd timesheet --minutes / streamd timesheet -m — HH:MM output

Closes #79

## Summary - Adds `-m`/`--minutes` flag to `streamd timesheet` command - When set, time is displayed as `HH:MM` (e.g., `8:30`) instead of decimal hours (e.g., `8.5h`) - Applies everywhere: per-day columns, monthly totals, cumulative balance, and outside-period warnings - Negative diffs display with a leading minus sign (e.g., `-1:30`) ## Changes - `src/cli/args.rs`: Converted `Timesheet` from unit variant to struct variant with `--minutes` / `-m` bool flag - `src/main.rs`: Destructures the new flag and passes it to `timesheet::run()` - `src/cli/commands/timesheet.rs`: Updated `format_hours()` and `format_diff()` to accept `use_minutes: bool`; propagated flag through all print helpers; added 8 unit tests ## Test plan - [ ] `cargo test` — all 164 tests pass - [ ] `cargo clippy` — no warnings - [ ] `streamd timesheet` — unchanged decimal output - [ ] `streamd timesheet --minutes` / `streamd timesheet -m` — HH:MM output Closes #79

kfickel added 1 commit 2026-04-07 07:58:00 +02:00

feat: add --minutes flag to timesheet command ... 42d9ecd3d9

Adds a -m/--minutes flag to `streamd timesheet` that displays time
in HH:MM format (e.g., 8:30) instead of decimal hours (e.g., 8.5h).
Includes unit tests for both formatting functions.

kfickel referenced this pull request 2026-04-07 07:58:11 +02:00

streamd timesheet report --use-minutes #79

kfickel added 1 commit 2026-04-07 07:59:57 +02:00

feat: bump to 0.2.0 b9954d2157

kfickel force-pushed 79_timesheet-use-minutes-flag from b9954d2157 to 9563ff4d93 2026-04-07 08:07:07 +02:00 Compare

kfickel added 1 commit 2026-04-07 08:09:54 +02:00

ci: fix release-pipeline running on feature-branches a9acd34801

kfickel merged commit a9acd34801 into main 2026-04-07 08:13:21 +02:00

kfickel deleted branch 79_timesheet-use-minutes-flag 2026-04-07 08:13:21 +02:00

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

planned

The feature is ready to be implemented by an AI agent

No labels

planned

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference: kfickel/streamd#82

No description provided.