Skip to content
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

fix: shortened the contributors list by filtering contributors according to their activity in that particular week. #194

Closed
wants to merge 1 commit into from

Conversation

sagarkori143
Copy link

fixes #171
Truncate the list of contributors on the basis of their activity(raised, merged, reviewed PRs and opened issues) in that particular week.

Copy link

vercel bot commented Dec 29, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
leaderboard ❌ Failed (Inspect) Dec 29, 2023 4:21am

@sagarkori143
Copy link
Author

sagarkori143 commented Dec 29, 2023

@rithviknishad I have done nothing with the data and contributors file (actually there were files and not folders with these names ). I did that in another branch and tested my changes and now I have made the required changes in a new branch without doing any ./load-fixtures.sh operation. Deployment failed is also the result of those files conflict just like i was not able to run local host because of them.

@rithviknishad
Copy link
Member

@sagarkori143 are you sure the deployment has failed because of these files?

Even if you had successfully ran load-fixtures.sh, it wouldn't affect because those files under data-repo directory (contributors and data) are git ignored.

The deployment failed because the code wasn't pretty
image

@sagarkori143
Copy link
Author

Okay let me just check on it once again but I thought that the deployment has failed just like it happened to me while running the localhost:
Screenshot 2023-12-29 100128

I'm looking for changes again.

@rithviknishad
Copy link
Member

@sagarkori143 closing this PR as #206 has fixed the underlying issue in a larger refactor and this PR was stale for a month with builds failing. Your contributions are appreciated.

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.

Truncate / exclude contributors that has 0 points in the weekly leaderboard page
2 participants