Skip to content

Conversation

@henriklundstrom
Copy link
Contributor

Implements the new endpoints for retrieving reports of copilot metrics for enterprises and organizations. See https://docs.github.com/en/enterprise-cloud@latest/rest/copilot/copilot-usage-metrics?apiVersion=2022-11-28. The API does not return the reports directly, but rather returns signed download URLs with a limited expiration time. The proposed implementation simply returns the API response, leaving the actual downloading of the reports to the user.

Will require an update to the openapi_operations.yaml-file.

@codecov
Copy link

codecov bot commented Dec 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.44%. Comparing base (c06e8a9) to head (70c494c).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3893      +/-   ##
==========================================
+ Coverage   92.39%   92.44%   +0.04%     
==========================================
  Files         203      203              
  Lines       14833    14925      +92     
==========================================
+ Hits        13705    13797      +92     
  Misses        926      926              
  Partials      202      202              

☔ 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.

@gmlewis
Copy link
Collaborator

gmlewis commented Dec 29, 2025

Will require an update to the openapi_operations.yaml-file.

Thank you, @henriklundstrom! I've updated the openapi_operations.yaml file in #3895.
If you merge master into this PR and push the result, then fix the linting errors, we should be ready to review this PR.

@henriklundstrom
Copy link
Contributor Author

Will require an update to the openapi_operations.yaml-file.

Thank you, @henriklundstrom! I've updated the openapi_operations.yaml file in #3895. If you merge master into this PR and push the result, then fix the linting errors, we should be ready to review this PR.

Thanks, I've merged the latest master into this branch.

@gmlewis gmlewis added the NeedsReview PR is awaiting a review before merging. label Dec 29, 2025
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, @henriklundstrom!
LGTM.
Awaiting second LGTM+Approval from any other contributor to this repo before merging.

cc: @stevehipwell - @alexandear - @zyfy29

@gmlewis gmlewis removed the NeedsReview PR is awaiting a review before merging. label Jan 2, 2026
@gmlewis
Copy link
Collaborator

gmlewis commented Jan 2, 2026

Thank you, @henriklundstrom and @alexandear!
Merging.

@gmlewis gmlewis merged commit 5dc0050 into google:master Jan 2, 2026
8 checks passed
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.

3 participants