Skip to content

Harvest rounder#110

Open
zimbatm wants to merge 2 commits intomainfrom
harvest-rounder
Open

Harvest rounder#110
zimbatm wants to merge 2 commits intomainfrom
harvest-rounder

Conversation

@zimbatm
Copy link
Member

@zimbatm zimbatm commented Dec 5, 2025

Automate the task of rounding hours to account for context switching
costs.

The main use-case is for people to export their own timesheets.
Automate the task of rounding hours to account for context switching
costs.
@zimbatm zimbatm requested a review from Mic92 December 5, 2025 11:26
};
packages = {
harvest-exporter = pkgs.callPackage ./harvest-exporter.nix { };
harvest-rounder = pkgs.callPackage ./harvest-rounder.nix { };
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't quiet understand yet, why this is a separate cli as opposed to a flag in the same script.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The exporter is read-only. I didn't want to mix it with a tool that writes back to Harvest.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But the harvest-exporter is not writing anything and it also uses the rounding that harvest reports?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The use-case is different. Here the goal is to update the recorded hours, with the rounding. The Harvest rounding has been turned off because it creates confusion with the customers when the reports don't match the invoiced hours.

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.

2 participants