Repository navigation
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. 📝 WalkthroughWalkthroughThe pull request updates pinned versions for one documentation dependency and eight development dependencies. The ChangesDependency Pin Updates
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: 🟡 Moderate · up to Python 3.9 contributors cannot set up the updated development environment. Provide compatible pins or version-specific requirements before merging. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 2 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1🧪 Generate unit tests (beta)
🛠️ Fix failing CI checks 💡
A rabbit checks the pins in rows, Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
- Configuration used: Organization UI
- Review profile: CHILL
- Plan: Advanced
- Run ID:
19178252-3cce-44e0-a9f1-2a93afa2a5ee
📒 Files selected for processing (2)
docs/requirements-docs.txtrequirements-dev.txt
Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.
📜 Review details
⏰ Context from checks skipped due to timeout. (8)
- GitHub Check: check-licenses
- GitHub Check: Build documentation
- GitHub Check: build (3.9)
- GitHub Check: build (3.11)
- GitHub Check: build (3.12)
- GitHub Check: build (3.10)
- GitHub Check: build (3.13)
- GitHub Check: Run linters
🔇 Additional comments (1)
docs/requirements-docs.txt (1)
7-7: LGTM!
| pytest==8.3.5 | ||
| respx==0.22.0 | ||
| setuptools==84.0.0 | ||
| pytest==9.1.1 |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
🔎 Supported by static analysis
🏁 Script executed:
#!/bin/bash
set -euo pipefail
rg -n --hidden \
-g 'pyproject.toml' -g 'setup.cfg' -g 'setup.py' -g 'tox.ini' -g 'noxfile.py' \
-g '.python-version' -g '*.yml' -g '*.yaml' \
'requires-python|python_requires|python-version|3\.9|py39' . || trueRepository: DagsHub/client
Length of output: 708
🏁 Script executed:
git diff --no-ext-diff --unified=3 996e58b84a201e59c6dff2832eb6068516796383 ead33fd8fb549443f5b98fb07e31026a6fda01bc -- requirements-dev.txt
printf '\n--- current file ---\n'
nl -ba requirements-dev.txt
printf '\n--- support declarations ---\n'
nl -ba setup.py | sed -n '66,78p'
nl -ba .github/workflows/python-package.yml | sed -n '12,30p'Repository: DagsHub/client
Length of output: 1924
🌐 Web query:
PyPI release metadata Requires-Python for pytest 9.1.1, pytest-env 1.7.1, fiftyone 1.22.1, datasets 5.0.1
💡 Result:
| PyPI release | `Requires-Python` |
|---|---|
| `pytest 9.1.1` | `>=3.10` ([pypi.org](https://pypi.org/project/pytest/9.1.1/)) |
| `pytest-env 1.7.1` | `>=3.10` ([pypi.org](https://pypi.org/project/pytest-env/1.7.1/)) |
| `fiftyone 1.22.1` | `>=3.10` ([pypi.org](https://pypi.org/project/fiftyone/?utm_source=openai)) |
| `datasets 5.0.1` | `>=3.10` ([pypi.org](https://pypi.org/project/datasets/?utm_source=openai)) |
Citations:
- 1: https://pypi.org/project/pytest/9.1.1/
- 2: https://pypi.org/project/pytest-env/1.7.1/
- 3: https://pypi.org/project/fiftyone/?utm_source=openai
- 4: https://pypi.org/project/datasets/?utm_source=openai
🌐 Web query:
PyPI exact release metadata Requires-Python fiftyone 1.22.1 datasets 5.0.1
💡 Result:
- **fiftyone 1.22.1:** `Requires-Python: >=3.10` ([pypi.org](https://pypi.org/project/fiftyone/1.22.1/))
- **datasets 5.0.1:** `Requires-Python: >=3.10.0` ([pypi.org](https://pypi.org/project/datasets/5.0.1/))
Citations:
- 1: https://pypi.org/project/fiftyone/1.22.1/
- 2: https://pypi.org/project/datasets/5.0.1/
Keep development dependencies compatible with Python 3.9.
The project supports Python 3.9, but pytest==9.1.1, pytest-env==1.7.1, fiftyone==1.22.1, and datasets==5.0.1 require Python 3.10 or later. Installing requirements-dev.txt on Python 3.9 therefore fails. Use Python 3.9-compatible releases or split the requirements by Python version and provide compatible alternatives.
|
Closing this in favor of #538 |
1 participant
Update sphinx-autodoc-typehints from 3.9.9 to 3.13.8.
Changelog
3.13.8
3.13.7
3.13.6
3.13.5
3.13.4
3.13.3
3.13.2
3.13.1
3.13.0
3.12.1
3.12.0
3.11.1
3.11.0
3.10.6
3.10.5
3.10.4
3.10.3
3.10.2
3.10.1
3.10.0
3.9.11
3.9.10
Links
Update setuptools from 80.9.0 to 84.0.0.
Changelog
84.0.0
83.0.0
82.0.1
82.0.0
81.0.0
80.10.2
80.10.1
80.10.0
Links
Update pytest from 8.3.5 to 9.1.1.
Changelog
9.1.1
9.1.0
9.0.3
9.0.2
9.0.1
9.0.0