Skip to content

Commit

Permalink
Drop the linters in the pip packager for newer CI linters
Browse files Browse the repository at this point in the history
Let's focus the pip packager to package building and unit testing
similarly to the rest of the packagers now that we have a separate
fully updated and extended CI lint workflow.
  • Loading branch information
pevogam committed Oct 23, 2024
1 parent 61fc44c commit 2279bc3
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 9 deletions.
6 changes: 0 additions & 6 deletions packaging/packager_pip.sh
Original file line number Diff line number Diff line change
Expand Up @@ -40,12 +40,6 @@ dnf -y install x11vnc
pip3 install --upgrade pip
pip3 install -r "$distro_root/guibot/packaging/pip_requirements.txt"

# auto-review and linters
cd "$distro_root/guibot"
echo "Performing auto-review linting checks on code and documentation"
pycodestyle guibot/* --ignore="E501,W503,E226,E265,E731,E306"
pydocstyle guibot/* --ignore="D212,D205,D400,D401,D415,D203,D105,D301,D302"

# pip packaging and installing of current guibot source
pip3 install wheel twine
cd "$distro_root/guibot/packaging"
Expand Down
3 changes: 0 additions & 3 deletions packaging/pip_requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,6 @@ Pyro4==4.82
# coverage analysis to use for testing
coverage
codecov
# linters and auto-reviews
pycodestyle
pydocstyle
# GUI to use for testing
# TODO: the most recent version at the time works on windows but not on linux
PyQt5==5.15.6; sys_platform == 'win32' and platform_python_implementation != "PyPy"
Expand Down

0 comments on commit 2279bc3

Please sign in to comment.