Skip to content

refactor: remove unused REPO_SCAN_* constants - #943

Closed
ebios-star wants to merge 1 commit into
entrius:testfrom
ebios-star:refactor/remove-unused-repo-scan-constants
Closed

refactor: remove unused REPO_SCAN_* constants#943
ebios-star wants to merge 1 commit into
entrius:testfrom
ebios-star:refactor/remove-unused-repo-scan-constants

Conversation

@ebios-star

Copy link
Copy Markdown
Contributor

Three constants in gittensor/constants.py are unreferenced anywhere in the codebase:

  • REPO_SCAN_PER_REPO_CAP
  • REPO_SCAN_GLOBAL_CAP
  • REPO_SCAN_CONCURRENCY

They were only used by gittensor/validator/issue_discovery/repo_scan.py, which was deleted in #796 when issue discovery moved to das-github-mirror. The constants were left orphaned.

Verified zero remaining references via grep -rni "REPO_SCAN" ..

Test suite: 726 passed.

These three constants were only referenced by gittensor/validator/issue_discovery/repo_scan.py, which was deleted in entrius#796 when issue discovery was routed through das-github-mirror. Verified zero remaining callers across the codebase.
@ebios-star

Copy link
Copy Markdown
Contributor Author

@anderdc whenever you have a moment, this is a small 5-line cleanup removing three constants left orphaned by #796. Happy to revise if you'd like a different scope. Thanks!

@xiao-xiao-mao xiao-xiao-mao Bot added the refactor Code restructuring without behavior change label May 3, 2026
@anderdc anderdc mentioned this pull request May 5, 2026
8 tasks
@anderdc

anderdc commented May 5, 2026

Copy link
Copy Markdown
Collaborator

Closing — please consolidate into #917.

@anderdc anderdc closed this May 5, 2026
@ebios-star

Copy link
Copy Markdown
Contributor Author

Consolidated into #955 per @anderdc's request.

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

Labels

refactor Code restructuring without behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants