URL: https://github.com/freeipa/freeipa/pull/1442
Author: tiran
Title: #1442: [Backport][ipa-4-6] Make fastlint even faster
Action: opened
PR body:
"""
This PR was opened automatically because PR #1438 was pushed to master and
backport to ipa-4-6 is required.
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/1442/head:pr1442
git checkout pr1442
From e155f8ea1ee7f491fa780366df7a349a141603c1 Mon Sep 17 00:00:00 2001
From: Christian Heimes <chei...@redhat.com>
Date: Fri, 5 Jan 2018 11:19:18 +0100
Subject: [PATCH] Make fastlint even faster
- Check pycodestyle before pylint. pycodestyle takes seconds while
pylint can easily take half a minute or more.
- Fix exit, needs two $
- Add some newlines to make output more readable
Signed-off-by: Christian Heimes <chei...@redhat.com>
---
Makefile.am | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index 85c020506f..1e200f117d 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -210,17 +210,17 @@ endif
FILES=$$(git diff --name-only $${MERGEBASE} \
| grep -E '\.py$$'); \
if [ -n "$${FILES}" ]; then \
- echo "Fast linting files: $${FILES}"; \
- echo "pylint"; \
- echo "------"; \
- PYTHONPATH=$(abspath $(top_srcdir)) $(PYTHON) -m pylint \
- --rcfile=$(top_srcdir)/pylintrc \
- --load-plugins pylint_plugins \
- $${FILES} || exit $?; \
+ echo -e "Fast linting files:\n$${FILES}\n"; \
echo "pycodestyle"; \
echo "-----------"; \
git diff $${MERGEBASE} | \
- $(PYTHON) -m pycodestyle --diff || exit $?; \
+ $(PYTHON) -m pycodestyle --diff || exit $$?; \
+ echo -e "\npylint"; \
+ echo "------"; \
+ PYTHONPATH=$(abspath $(top_srcdir)) $(PYTHON) -m pylint \
+ --rcfile=$(top_srcdir)/pylintrc \
+ --load-plugins pylint_plugins \
+ $${FILES} || exit $$?; \
else \
echo "No modified Python files found"; \
fi
_______________________________________________
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org
