gemini-code-assist[bot] commented on code in PR #18777:
URL: https://github.com/apache/tvm/pull/18777#discussion_r2807087137
##########
docker/install/ubuntu_install_python_package.sh:
##########
@@ -29,17 +29,17 @@ pip3 install --upgrade \
numpy==1.21.* \
orderedset \
packaging \
- Pillow==9.1.0 \
+ Pillow \
psutil \
pytest \
git+https://github.com/tlc-pack/tlcpack-sphinx-addon.git@768ec1dce349fe4708f6ad68be1ebb3f3dabafa1
\
pytest-profiling \
pytest-xdist \
- pytest-rerunfailures==10.2 \
+ pytest-rerunfailures \
requests \
scipy \
Jinja2 \
- junitparser==2.4.2 \
+ junitparser \
Review Comment:
Unpinning dependencies can lead to non-reproducible builds and make it
difficult to debug CI failures caused by upstream changes. To ensure build
stability, it's a best practice to pin dependencies to specific versions.
Instead of removing the version pins for `Pillow`, `pytest-rerunfailures`,
and `junitparser`, could you please update these packages to their latest
stable versions and pin them? This will provide the benefits of newer versions
while keeping the CI environment deterministic.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
