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

total_repositories in DoT must ignore archived repos #239

Closed
yegor256 opened this issue Aug 7, 2024 · 4 comments · Fixed by #371
Closed

total_repositories in DoT must ignore archived repos #239

yegor256 opened this issue Aug 7, 2024 · 4 comments · Fixed by #371
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@yegor256
Copy link
Member

yegor256 commented Aug 7, 2024

total_repositories in dimensions-of-terrain.rb must ignore archived repos, don't count them.

@yegor256 yegor256 added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Aug 7, 2024
@Yegorov
Copy link
Contributor

Yegorov commented Sep 30, 2024

@yegor256 Method Fbe.unmask_repos already rejects archival repositories, based on zerocracy/fbe#46
Or need to duplicate the check here?

@yegor256
Copy link
Member Author

yegor256 commented Oct 1, 2024

@Yegorov it's better to do an additional check here

@Yegorov
Copy link
Contributor

Yegorov commented Oct 2, 2024

@yegor256 may I take this one?

@yegor256
Copy link
Member Author

yegor256 commented Oct 3, 2024

@Yegorov go ahead

Yegorov added a commit to Yegorov/judges-action that referenced this issue Oct 3, 2024
yegor256 added a commit that referenced this issue Oct 4, 2024
#239: ignore archived repos when count `total_repositories` in DoT
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants