-
Notifications
You must be signed in to change notification settings - Fork 955
Add Clones metric API (#2604) #3247
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
358ab5a to
2cac481
Compare
|
Hi! This is my first contribution to CHAOSS. Please let me know if any changes are required. Looking forward to your feedback! |
40596f8 to
5e7c270
Compare
Ulincsys
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are extra changes in this PR that should not be included.
Please rebase and remove the extra changes from each PR, and I can review them independently.
|
|
||
| owner, name = get_owner_repo(repo.repo_git) | ||
| url = f"https://api.github.com/repos/{owner}/{name}" | ||
| try: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your error handling changes were included in this PR, and also #3248
|
Sure, i’ll make the changes in a while , i am having my exams right now.
…On Mon, Sep 15, 2025 at 9:27 AM John McGinness ***@***.***> wrote:
***@***.**** requested changes on this pull request.
There are extra changes in this PR that should not be included.
Please rebase and remove the extra changes from each PR, and I can review
them independently.
—
Reply to this email directly, view it on GitHub
<#3247 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BCCKHZZBKAL257YKR6AR4Z33SY2LRAVCNFSM6AAAAACCZY7X2KVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTEMRSHAZTSMRYG4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
5e7c270 to
2cac481
Compare
Signed-off-by: saksham23467 <142910439+saksham23467@users.noreply.github.com>
2cac481 to
cd87f5c
Compare
|
This has been sitting here for a while and we are going through things ... it does not look like its been tested, and its pretty isolated from our other code ... AND, these metrics are NOT USEFUL UNLESS YOU ARE USING AN API KEY that has maintainer rights on the repos you are getting data for. So, if you are running Augur only on the repos you have those privileges on you will get this data ... most OSPO driven Augur instances will not be able to capture this data. |
Description
This PR implements the Clones metric API as defined by CHAOSS. It returns the number of repository clones (both total and unique) for a given repository or repo group, filtered by an optional time range.
Fixes
Fixes #2604
Notes for Reviewers
augur_data.repo_clones_datatable.repo_id,repo_group_id,begin_date, andend_date.Signed commits