[Bug report] Unspecified pylint version in client-python #3558
Assignees
Labels
bug
Something isn't working
Comments
jerryshao
pushed a commit
that referenced
this issue
May 29, 2024
### What changes were proposed in this pull request? As title, specify pylint version in order to avoid unexpected bugs in the future development. ### Why are the changes needed? Fix: #3558 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? `./gradlew build -x test` or `./gradlew :clients:client-python:pylint` for for the convenience.
github-actions bot
pushed a commit
that referenced
this issue
May 29, 2024
### What changes were proposed in this pull request? As title, specify pylint version in order to avoid unexpected bugs in the future development. ### Why are the changes needed? Fix: #3558 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? `./gradlew build -x test` or `./gradlew :clients:client-python:pylint` for for the convenience.
jerryshao
added a commit
that referenced
this issue
May 29, 2024
### What changes were proposed in this pull request? As title, specify pylint version in order to avoid unexpected bugs in the future development. ### Why are the changes needed? Fix: #3558 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? `./gradlew build -x test` or `./gradlew :clients:client-python:pylint` for for the convenience. Co-authored-by: Liang Chun <[email protected]>
diqiu50
pushed a commit
to diqiu50/gravitino
that referenced
this issue
Jun 13, 2024
…pache#3559) ### What changes were proposed in this pull request? As title, specify pylint version in order to avoid unexpected bugs in the future development. ### Why are the changes needed? Fix: apache#3558 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? `./gradlew build -x test` or `./gradlew :clients:client-python:pylint` for for the convenience.
coolderli
pushed a commit
that referenced
this issue
Dec 17, 2024
### What changes were proposed in this pull request? As title, specify pylint version in order to avoid unexpected bugs in the future development. ### Why are the changes needed? Fix: #3558 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? `./gradlew build -x test` or `./gradlew :clients:client-python:pylint` for for the convenience. (cherry picked from commit 63592af)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Version
main branch
Describe what's wrong
Pylint will generate error that can when their version is outdated. For example, in my local environment with pylint version = 2.8.2, which is not the same with what runs in github action (pylint = 3.1.0), therefore cause build error in local env.
Error message and/or stacktrace
How to reproduce
In gravitino
0.5.0, run commend./gradlew :clients:client-python:pylintor./gradlew build -x testwith pylint version that is not3.1.0Additional context
Maybe other package version should be specified, either.
The text was updated successfully, but these errors were encountered: