Skip to content

Commit

Permalink
Remove old Makefile targets.
Browse files Browse the repository at this point in the history
- `pep8` - There is no need to give it more prominent placement than other sanity tests.
- `pyflakes` - The project only makes use of `pylint` and `pycodestyle`. Having the target implies that it should work.
- `loc` - Not used. Easily run manually if needed.
  • Loading branch information
mattclay committed Aug 14, 2019
1 parent 737a500 commit f0d32a2
Showing 1 changed file with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
# make deb ------------------ produce a DEB
# make docs ----------------- rebuild the manpages (results are checked in)
# make tests ---------------- run the tests (see https://docs.ansible.com/ansible/devel/dev_guide/testing_units.html for requirements)
# make pyflakes, make pep8 -- source code checks

########################################################
# variable section
Expand Down Expand Up @@ -172,18 +171,6 @@ authors:
%.1: %.1.rst lib/ansible/release.py
$(ASCII2MAN)

.PHONY: loc
loc:
sloccount lib library bin

.PHONY: pep8
pep8:
$(ANSIBLE_TEST) sanity --test pep8 --python $(PYTHON_VERSION) $(TEST_FLAGS)

.PHONY: pyflakes
pyflakes:
pyflakes lib/ansible/*.py lib/ansible/*/*.py bin/*

.PHONY: clean
clean:
@echo "Cleaning up distutils stuff"
Expand Down

0 comments on commit f0d32a2

Please sign in to comment.