Skip to content

Conversation

@Noaman-Akhtar
Copy link
Contributor

Description

  • Removed redundant TODO comment on line 114 (logic was already implemented)
  • Removed question TODO on line 153 (no need to check, UserRepo.insert handles duplicates)
  • Removed TODO on line 201 (caller already chunks data into 100)
  • Deleted ~40 lines of dead commented code at end of file

This PR fixes #3574

Notes for Reviewers
All TODOs were either already implemented or pointed to dead code that was safe to remove.

Signed commits

  • Yes, I signed my commits.

yield data[i:i + size]


# TODO: Make it only get like 100 at a time
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Noaman-Akhtar Where was this implemented?

Copy link
Contributor

@guptapratykshh guptapratykshh Jan 14, 2026

Choose a reason for hiding this comment

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

this is implemented in the function - add_github_orgs_and_repos on line 59

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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.

address and cleanup todo comments in frontend.py

3 participants