Skip to content

Conversation

@tomfeigin
Copy link
Contributor

@tomfeigin tomfeigin commented Dec 7, 2025

BREAKING CHANGE: UsageItem.Quantity is now type float64.

The organization usage report schema expected the 'Quantity' field to be a float.
This PR fixes the decoding of the usage report by conforming to the GitHub API schema

@gmlewis gmlewis changed the title fix: org usage report response decode fix!: Change Org usage report Quantity to float64 Dec 7, 2025
@gmlewis gmlewis added NeedsReview PR is awaiting a review before merging. Breaking API Change PR will require a bump to the major version num in next release. Look here to see the change(s). labels Dec 7, 2025
@codecov
Copy link

codecov bot commented Dec 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.36%. Comparing base (66f826c) to head (8cd9e34).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3862   +/-   ##
=======================================
  Coverage   92.36%   92.36%           
=======================================
  Files         199      199           
  Lines       14308    14308           
=======================================
  Hits        13215    13215           
  Misses        894      894           
  Partials      199      199           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @tomfeigin!
LGTM.
Awaiting second LGTM+Approval from any other contributor to this repo before merging.

cc: @stevehipwell - @alexandear - @zyfy29

Copy link
Contributor

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

LGTM

@gmlewis gmlewis removed the NeedsReview PR is awaiting a review before merging. label Dec 8, 2025
@gmlewis
Copy link
Collaborator

gmlewis commented Dec 8, 2025

Thank you, @stevehipwell!
Merging.

@gmlewis gmlewis merged commit 95c2a13 into google:master Dec 8, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Breaking API Change PR will require a bump to the major version num in next release. Look here to see the change(s).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants