Skip to content

Refactor export_summary for configurability and validation#131

Merged
NessieCanCode merged 1 commit intomainfrom
refactor-export_summary-for-robustness-and-readability
Sep 9, 2025
Merged

Refactor export_summary for configurability and validation#131
NessieCanCode merged 1 commit intomainfrom
refactor-export_summary-for-robustness-and-readability

Conversation

@NessieCanCode
Copy link
Owner

Summary

  • Parameterize rates file path and validate cluster core count
  • Split export_summary logic into typed helper methods for loading rates, cost aggregation, time series, and projected revenue
  • Cover new validation and helpers with updated unit tests

Testing

  • PYTHONPATH=src python test/unit/billing_summary.test.py

https://chatgpt.com/codex/tasks/task_e_68bfa008948c832484866710961f0e10

@NessieCanCode NessieCanCode merged commit 7859d02 into main Sep 9, 2025
0 of 4 checks passed
@NessieCanCode NessieCanCode deleted the refactor-export_summary-for-robustness-and-readability branch September 9, 2025 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant