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.

Sign up for GitHub

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

Jump to bottom

tidy: remove ENTRY_LIMIT maximum checking and set it to 900 #110970

Merged
merged 1 commit into from
May 4, 2023
Merged

tidy: remove ENTRY_LIMIT maximum checking and set it to 900 #110970

merged 1 commit into from
May 4, 2023
+2 −5

Commits on Apr 29, 2023

  1. Remove ENTRY_LIMIT maximum checking and set it to 900 

    The number 900 is safely below github's limit of 1000 entries for a directory.
PRs to move tests can still decrease the sizes of various directories,
but adjusting the limit won't be neccessary any more.

In general, the limit is a bad tool to direct people to put tests into
fitting directories because when those are available, usually the limit
is not hit, while the limit is hit in directories that have a weak
substructure themselves.
    @est31
    est31 committed Apr 29, 2023
    Configuration menu
    Copy the full SHA
    e498402 View commit details
    Browse the repository at this point in the history