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

Rely on the Linguist::Repository::MAX_TREE_SIZE default value more #6830

Merged
merged 2 commits into from
May 15, 2024

Conversation

issyl0
Copy link
Collaborator

@issyl0 issyl0 commented May 15, 2024

Description

  • Changing it won't be such a problem in future if there's just one place to change it. I hadn't realised it had propagated so much!

Checklist:

  • I am adding new or changing current functionality
    • I have added or updated the tests for the new or changed functionality.

- Changing it won't be such a problem in future if there's just one place to change it. I hadn't realised it had propagated so much!
@issyl0 issyl0 requested a review from a team as a code owner May 15, 2024 11:06
@issyl0 issyl0 changed the title Rely on the Linguist::Repository::MAX_TREE_SIZE default value more Rely on the Linguist::Repository::MAX_TREE_SIZE default value more May 15, 2024
@lildude lildude added this pull request to the merge queue May 15, 2024
Merged via the queue into master with commit e2012cd May 15, 2024
8 checks passed
@lildude lildude deleted the use-max-tree-size-constant-not-magic-number branch May 15, 2024 15:00
@github-linguist github-linguist deleted a comment from DobahHk May 17, 2024
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.

None yet

2 participants