Skip to content

fix: correct project details handling for forked repository - #382

Open
trishanu-init wants to merge 1 commit into
interviewstreet:mainfrom
trishanu-init:feature/correct-github-repo-details
Open

fix: correct project details handling for forked repository#382
trishanu-init wants to merge 1 commit into
interviewstreet:mainfrom
trishanu-init:feature/correct-github-repo-details

Conversation

@trishanu-init

Copy link
Copy Markdown

Pull Request: Fix Project details handling for forked repositories

Related Issues

Closes #162

Summary

This PR resolves following issue in github.py:

  1. Repository Filtering & Fork Handling (Bug: Project details of forked repository are not accounted for and skipped from all_repositories because of false positive of forks_count<5 #162)
    • Removed unnecessary filtering of forked repositories with low forks_count.
    • Ensures data is fetched from the parent repository when available.
    • Adds fallback to child repo details if parent data fails.
    • Corrects inaccurate metrics for forked open-source projects.

Impact

  • Accurate and complete repository metadata.
  • Improved open-source project evaluation reliability.
  • Fixes false scoring of candidate due to incorrect project details.

Testing

image

Current (fix)
image

@trishanu-init

Copy link
Copy Markdown
Author

Hi @sp2hari , Can you please review this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Project details of forked repository are not accounted for and skipped from all_repositories because of false positive of forks_count<5

1 participant