Skip to content

Commit

Permalink
Fix lint does not work with astroid==3.0 error. (#496)
Browse files Browse the repository at this point in the history
`python3 -m pip install --upgrade prospector pip` has a change to
install astroid==3.0 (do not know why) which is not compatible with
lint.

Fix astroid version to 2.15.7 in this PR.



By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.

Co-authored-by: Xiang Song <[email protected]>
  • Loading branch information
classicsong and Xiang Song authored Sep 27, 2023
1 parent b19bdcf commit 58262c6
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions .github/workflow_scripts/lint_check.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@ cd ../../
set -ex

python3 -m pip install --upgrade prospector pip
pip3 uninstall -y astroid
yes | pip3 install astroid==2.15.7
FORCE_CUDA=1 python3 -m pip install -e '.[test]' --no-build-isolation
pylint --rcfile=./tests/lint/pylintrc ./python/graphstorm/data/*.py
pylint --rcfile=./tests/lint/pylintrc ./python/graphstorm/dataloading/
Expand Down

0 comments on commit 58262c6

Please sign in to comment.