Skip to content

Commit

Permalink
Update get-contributors-data.js
Browse files Browse the repository at this point in the history
Added third main update to consider comments if the `created_at` date is recent (in order to avoid a minimizing of a comment being considered as an update)
  • Loading branch information
t-will-gillis authored Oct 28, 2024
1 parent 5c80b63 commit e889ac3
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -105,8 +105,8 @@ async function fetchContributors(dates){
if(contributorInfo.author){
allContributorsSince[contributorInfo.author.login] = true;
}
// Check for username in `user.login`, but skip `user.login` covered by 3rd API
else if(contributorInfo.user && api != 'GET /repos/{owner}/{repo}/issues'){
// Check for username in `user.login`, check only comment's `created_at` time, and skip `user.login` covered by 3rd API
else if(contributorInfo.user && contributorInfo.created_at > date && api != 'GET /repos/{owner}/{repo}/issues'){
allContributorsSince[contributorInfo.user.login] = true;
}
// This check is done for `/issues` (3rd) API. Sometimes a user who created an issue is not the same as the
Expand Down

0 comments on commit e889ac3

Please sign in to comment.