Change total_minutes_used and included_minutes from int to float64#2643
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2643 +/- ##
=======================================
Coverage 98.05% 98.05%
=======================================
Files 130 130
Lines 11242 11242
=======================================
Hits 11023 11023
Misses 150 150
Partials 69 69
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
gmlewis
left a comment
There was a problem hiding this comment.
Thank you, @loispostula !
LGTM.
Awaiting second LGTM+Approval from any other contributor to this repo.
|
Thank you, @valbeat ! |
|
@gmlewis np, any info regarding the next release date? |
The last release was 2 weeks ago, and we have a potentially broad-reaching breaking API change coming up shortly: #2645 Are you OK with that, @loispostula ? |
|
I'll continue using my fork while waiting ;) |
Fixes #2642
GitHub Actions returns total_minutes_used and included_minutes as floating point value.
The Implementation should accept this floating point value as float64.
Example
{ "total_minutes_used": 1234.0, "total_paid_minutes_used": 3959.0, "included_minutes": 3000.0, "minutes_used_breakdown": { "UBUNTU": 1234, "MACOS": 1234, "WINDOWS": 0 } }