Skip to content

Add Copilot Usage Summary for Organization - #3318

Closed
claystation wants to merge 4 commits into
google:masterfrom
nordnet:copilot_usage
Closed

Add Copilot Usage Summary for Organization#3318
claystation wants to merge 4 commits into
google:masterfrom
nordnet:copilot_usage

Conversation

@claystation

Copy link
Copy Markdown
Contributor

Github has a Rest API to get the usage summary of Copilot within the organization. This PR implements the functionality to fetch that data.
API Docs: Here

Please let me know what you think!

@google-cla

google-claBot commented Oct 9, 2024

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@claystationclaystation changed the title Copilot Organization SummaryCopilot Usage Summary for OrganizationOct 9, 2024
@gmlewisgmlewis changed the title Copilot Usage Summary for OrganizationAdd Copilot Usage Summary for OrganizationOct 9, 2024
@gmlewisgmlewis added NeedsReview PR is awaiting a review before merging. waiting for signed CLA labels Oct 9, 2024
@codecov

codecovBot commented Oct 9, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.06%. Comparing base (2b8c7fa) to head (d45f521).
Report is 144 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@## master #3318 +/- ##
==========================================
- Coverage 97.72% 93.06% -4.67% 
==========================================
Files 153 172 +19 Lines 13390 11773 -1617 ==========================================
- Hits 13085 10956 -2129 - Misses 215 723 +508 - Partials 90 94 +4 

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

@gmlewisgmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @claystation !
Just a few tweaks, please, then we should be ready for a second LGTM+Approval from any other contributor to this repo (and signed CLA) before merging.

Comment threadgithub/copilot.go
Comment threadgithub/copilot.go
Comment threadgithub/copilot.go
Comment threadgithub/copilot_test.go
@claystation

Copy link
Copy Markdown
ContributorAuthor

@gmlewis i will close this PR due to CLA signing issues. I opened #3321 which also contains a commit with the requested changes. Please review that one, i hope it should be good!

@gmlewisgmlewis removed NeedsReview PR is awaiting a review before merging. waiting for signed CLA labels Oct 10, 2024
Sign up for freeto 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.

4 participants

@claystation@gmlewis@stasostrovskyi@henriklundstrom