Skip to content

♻️ Update the lru_cache limit for dependencies to account for large apps#16062

Merged
tiangolo merged 1 commit into
masterfrom
marmemlo
Jul 27, 2026
Merged

♻️ Update the lru_cache limit for dependencies to account for large apps#16062
tiangolo merged 1 commit into
masterfrom
marmemlo

Conversation

@tiangolo

Copy link
Copy Markdown
Member

Pull Request

♻️ Update the lru_cache limit for dependencies to account for large apps

Some users reported a number of dependencies larger than 1024, this should account for larger apps. I'll wait to see if anyone requests an even larger bump, for now this should work.

Discussion:

Description

AI Disclaimer

Codex with gpt-5.6-sol. Based on feedback from users on #16049 (comment) and shared in private messages.

AI transcript

Checklist

  • This PR links to a GitHub Discussion for the proposed code change.
  • I added tests for the change.
  • The new or updated tests fail on the main branch and pass on this PR.
  • Coverage stays at 100%.
  • The documentation explains the change if needed.

@tiangolo
tiangolo marked this pull request as ready for review July 27, 2026 11:15
@codspeed-hq

codspeed-hq Bot commented Jul 27, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 21 untouched benchmarks


Comparing marmemlo (d7d262b) with master (255b912)

Open in CodSpeed

@tiangolo
tiangolo enabled auto-merge (squash) July 27, 2026 11:17
@tiangolo
tiangolo merged commit 65ef53a into master Jul 27, 2026
42 of 43 checks passed
@tiangolo
tiangolo deleted the marmemlo branch July 27, 2026 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant