Skip to content

Conversation

sandy3w
Copy link
Member

@sandy3w sandy3w commented Aug 30, 2025

Addresses #14:
** changes made to repo labeler files are the same as the other PR I put in
** 8/29: I made this PR as I will not be available for the upcoming 1+ week , please ignore or close this PR if it is not ready yet!!

This PR includes two sets of files:
Set 1:
repo_fetcher:

  • poc_results.csv

  • repo_fetcher.py

  • target_repo.json

  • repo_fetcher.yml

Set 2:
org_fetcher:

  • org_contr.csv

  • org_fetcher.py

  • target_org.json

  • org_fetcher.yml

Set 1:
Set 1 contains the files related to the POC I used. The files work as following: repo_fetcher.yml is run as a GitHub action -> calls repo_fetcher.py -> which reads target_repo.json and outputs to poc_results.csv

For target_repo.json, repo name is mandatory while username is not.

This set is not needed as all functionalities are also in set 2.

Set 2:
Set 2 contains the finished scripted. The files work as following: org_fetcher.yml is run as a GitHub action -> calls org_fetcher.py -> which reads target_org.json and outputs to porg_contr.csv

For target_org.json, only repo is mandatory, meaning user and repo can be left as empty lists and the script will fetch for all (e.g. if user is not specified but repo is, script will fetch all contributions made to the repo).

The output csv is stored as an artifact.

@sandy3w sandy3w requested a review from chinaexpert1 August 30, 2025 03:36
Copy link
Member

@chinaexpert1 chinaexpert1 left a comment

Choose a reason for hiding this comment

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

I see I have open comments here Sandy have you done them or do you need another week?

@sandy3w
Copy link
Member Author

sandy3w commented Sep 11, 2025

@chinaexpert1 Thanks for the reviews! I've made changes accordingly and coordinated the changes to be made on both sets of files. Let me know if there is anymore issues:)

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.

2 participants