Skip to content

Commit

Permalink
chore: disable pylint running on main branch (#2041)
Browse files Browse the repository at this point in the history
* fix: disable pylint running on main branch

Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>

* fix: address MANIFEST.in

Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>

* Update MANIFEST.in

Co-authored-by: Chaoyu <paranoyang@gmail.com>
  • Loading branch information
aarnphm and parano committed Nov 22, 2021
1 parent dc17592 commit 31f5f3a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 20 deletions.
13 changes: 3 additions & 10 deletions MANIFEST.in
@@ -1,13 +1,6 @@
include README.md
include LICENSE

# Included in PyPI distribution
include bentoml/*.py
include bentoml/_version.py
include bentoml/_internal/bento/docker-entrypoint.sh
graft bentoml/_internal
graft bentoml
# All files tracked by Git are included in PyPI distribution

# Files to exclude in PyPI distribution
exclude .gitignore
exclude .readthedocs.yml
exclude .pep8speaks.yml
Expand All @@ -20,7 +13,7 @@ exclude SECURITY.md
exclude Makefile
exclude MANIFEST.in

# Prune directory not to be included in PyPI package
# Directories to exclude in PyPI package
prune requirements
prune tests
prune docker
Expand Down
10 changes: 0 additions & 10 deletions scripts/ci/style/pylint_check.sh
Expand Up @@ -15,16 +15,6 @@ if [[ -n "$GITHUB_BASE_REF" ]]; then
FAIL "pylint failed."
exit 1
fi
else
echo "Running pylint for the whole library..."
pylint --rcfile="$GIT_ROOT/pyproject.toml" bentoml | tee /tmp/"$name"_bentoml
pylint --rcfile="$GIT_ROOT/pyproject.toml" --disable=E0401,F0010 tests docker | tee /tmp/"$name"_tests_docker

if [ -s /tmp/"$name"_tests_docker ] || [ -s /tmp/"$name"_bentoml ]; then
FAIL "pylint failed"
cat /tmp/"$name"_tests_docker || cat /tmp/"$name"_bentoml
exit 1
fi
fi

PASS "pylint check passed!"
Expand Down

0 comments on commit 31f5f3a

Please sign in to comment.