Enable Travis-CI

- use /usr/bin/env instead of hard-coded /usr/bin/python3
- move mypy checks into the `check` target
- disable pylint import checks for setup.py because of
  pylint-dev/pylint#73 (comment)

Author: atodorov

.travis.yml

@@ -0,0 +1,11 @@
+language: python
+python:
+  - "3.5"
+
+install:
+  - pip install --upgrade pip setuptools
+  - pip install coverage mypy-lang nose ordered-set polib pylint requests six
+  - pip install https://github.com/atodorov/pocketlint/zipball/fallback_to_pypi_pylint
+
+script:
+  - make ci

Makefile

@@ -46,9 +46,12 @@ docs:
 check:
 	@echo "*** Running pylint to verify source ***"
 	PYTHONPATH=. tests/pylint/runpylint.py
+	@which mypy || (echo "*** Please install mypy (python3-mypy) ***"; exit 2)
+	@echo "*** Running type checks ***"
+	PYTHONPATH=. mypy pykickstart
 	@echo "*** Running tests on translatable strings ***"
 	$(MAKE) -C po $(PKGNAME).pot
-	PYTHONPATH=translation-canary python3 -m translation_canary.translatable po/$(PKGNAME).pot
+	PYTHONPATH=translation-canary $(PYTHON) -m translation_canary.translatable po/$(PKGNAME).pot
 	git checkout -- po/$(PKGNAME).pot || true
 
 # Left here for backwards compability - in case anyone was running the test target.  Now you always get coverage.
@@ -59,10 +62,7 @@ coverage:
 	@echo "*** Running unittests with coverage ***"
 	PYTHONPATH=. $(PYTHON) -m nose --with-coverage --cover-erase --cover-branches --cover-package=pykickstart --cover-package=tools $(NOSEARGS)
 	$(COVERAGE) combine
-	$(COVERAGE) report -m | tee coverage-report.log
-	@which mypy || (echo "*** Please install mypy (python3-mypy) ***"; exit 2)
-	@echo "*** Running type checks ***"
-	PYTHONPATH=. mypy pykickstart
+	$(COVERAGE) report -m --include="pykickstart/*,tools/*" | tee coverage-report.log
 
 clean:
 	-rm *.tar.gz pykickstart/*.pyc pykickstart/*/*.pyc tests/*.pyc tests/*/*.pyc docs/programmers-guide *log .coverage
@@ -87,7 +87,7 @@ archive: check test tag docs
 	cp -r po/*.po $(PKGNAME)-$(VERSION)/po/
 	$(MAKE) -C $(PKGNAME)-$(VERSION)/po
 	cp docs/programmers-guide $(PKGNAME)-$(VERSION)/docs/
-	PYTHONPATH=translation-canary python3 -m translation_canary.translated --release $(PKGNAME)-$(VERSION)
+	PYTHONPATH=translation-canary $(PYTHON) -m translation_canary.translated --release $(PKGNAME)-$(VERSION)
 	( cd $(PKGNAME)-$(VERSION) && $(PYTHON) setup.py -q sdist --dist-dir .. )
 	rm -rf $(PKGNAME)-$(VERSION)
 	git checkout -- po/$(PKGNAME).pot
@@ -141,6 +141,6 @@ rc-release: scratch-bumpver scratch
 	mock -r $(MOCKCHROOT) --rebuild *src.rpm --resultdir $(shell pwd)  || exit 1
 
 ci:
-	$(MAKE) PYTHON=python3 check coverage
+	$(MAKE) PYTHON=$(PYTHON) check coverage
 
 .PHONY: check clean install tag archive local docs release

setup.py

@@ -1,3 +1,5 @@
+# pylint: disable=import-error,no-name-in-module
+# See https://github.com/PyCQA/pylint/issues/73#issuecomment-163171888
 from distutils.core import setup
 from distutils.command.install_scripts import install_scripts as _install_scripts
 from distutils.file_util import move_file

tests/pylint/runpylint.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python3
+#!/usr/bin/env python3
 
 import sys
 

tools/ksflatten.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python3
+#!/usr/bin/env python3
 #
 # Simple script to take a kickstart config, read it in, parse any %includes,
 # etc to write out a flattened config that is stand-alone

tools/ksshell.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python3
+#!/usr/bin/env python3
 #
 # Chris Lumens <[email protected]>
 #

tools/ksvalidator.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/env python3
 #
 # Chris Lumens <[email protected]>
 #

tools/ksverdiff.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python3
+#!/usr/bin/env python3
 #
 # Chris Lumens <[email protected]>
 #

translation-canary/tests/pylint/runpylint.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python3
+#!/usr/bin/env python3
 
 import sys
 from pocketlint import PocketLintConfig, PocketLinter