chore: move tqdm to development dependencies - #1525
Conversation
Signed-off-by: Harrison King Saturley-Hall <hsaturleyhal@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📜 Recent review details🧰 Additional context used📓 Path-based instructions (1)**/*⚙️ CodeRabbit configuration file
Files:
🔇 Additional comments (1)
WalkthroughThe project removes ChangesDependency classification
Estimated code review effort: 1 (Trivial) | ~2 minutes Mergeability Score: ⚪ Minimal · up to This change removes tqdm from regular installations while retaining it for development environments; no actionable merge-blocking risk remains after normal checks and review. Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
Comment |
Summary
tqdmfrom the default installation requirements to thedevextrauv.locksotqdmis selected byextra == 'dev'Why
AIConfigurator's packaged CLI and SDK do not import
tqdm. Its repository uses are limited to Collector and development tools, whose documented and CI environments already install thedevextra.Regular
aiconfiguratorinstallations will no longer receivetqdm; development and test installations remain unchanged.Validation
uv lock --checktqdmdevexport includestqdm==4.67.3Requires-Dist: tqdm>=4.0.0; extra == "dev"git diff --checkSummary by CodeRabbit