meta: Rename virtualenv, add check target (#183)

* meta: Rename virtualenv, add check target

* fix: Remove useless arg from chown

* fix: Ignore new venv path in flake8

Author: untitaker

.flake8

@@ -3,4 +3,4 @@ ignore = E203, E266, E501, W503, E402, E731
 max-line-length = 80
 max-complexity = 18
 select = B,C,E,F,W,T4,B9
-exclude=venv,.eggs
+exclude=.venv,.eggs

Makefile

@@ -26,7 +26,7 @@ doc:
 	@cargo doc
 .PHONY: doc
 
-test: cargotest pytest integration-test
+test: cargotest pytest
 .PHONY: test
 
 cargotest: GeoLite2-City.mmdb
@@ -54,29 +54,29 @@ pytest:
 	@$(MAKE) -C py test
 .PHONY: pytest
 
-venv/bin/python: Makefile
-	rm -rf venv
-	virtualenv -p $$SEMAPHORE_PYTHON_VERSION venv
+.venv/bin/python: Makefile
+	rm -rf .venv
+	virtualenv -p $$SEMAPHORE_PYTHON_VERSION .venv
 
-integration-test: venv/bin/python
-	venv/bin/pip install -U pytest pytest-localserver requests flask "sentry-sdk>=0.2.0" pytest-rerunfailures pytest-xdist "git+https://github.com/untitaker/pytest-sentry#egg=pytest-sentry"
+integration-test: .venv/bin/python
+	.venv/bin/pip install -U pytest pytest-localserver requests flask "sentry-sdk>=0.2.0" pytest-rerunfailures pytest-xdist "git+https://github.com/untitaker/pytest-sentry#egg=pytest-sentry"
 	cargo build
-	@venv/bin/pytest tests -n12 --reruns 5
+	@.venv/bin/pytest tests -n12 --reruns 5
 .PHONY: integration-test
 
-python-format: venv/bin/python
-	venv/bin/pip install -U black
-	venv/bin/black .
+python-format: .venv/bin/python
+	.venv/bin/pip install -U black
+	.venv/bin/black .
 .PHONY: python-format
 
-python-format-check: venv/bin/python
-	venv/bin/pip install -U black
-	venv/bin/black --check .
+python-format-check: .venv/bin/python
+	.venv/bin/pip install -U black
+	.venv/bin/black --check .
 .PHONY: python-format
 
-python-lint: venv/bin/python
-	venv/bin/pip install -U flake8
-	venv/bin/flake8
+python-lint: .venv/bin/python
+	.venv/bin/pip install -U flake8
+	.venv/bin/flake8
 .PHONY: python-format
 
 format: python-format
@@ -97,6 +97,9 @@ clippy:
 lint: clippy python-lint
 .PHONY: lint
 
+check: format test
+.PHONY: check
+
 test-process-event:
 	# Process a basic event and assert its output
 	bash -c 'diff \

py/Makefile

@@ -1,26 +1,26 @@
-venv:
-	@$(MAKE) -C .. venv/bin/python
-.PHONY: venv
+.venv:
+	@$(MAKE) -C .. .venv/bin/python
+.PHONY: .venv
 
-install: venv
-	SEMAPHORE_DEBUG=1 ../venv/bin/pip install -v --editable .
+install: .venv
+	SEMAPHORE_DEBUG=1 ../.venv/bin/pip install -v --editable .
 .PHONY: install
 
 test: install
-	../venv/bin/pip install -U pytest
-	@../venv/bin/pytest tests
+	../.venv/bin/pip install -U pytest
+	@../.venv/bin/pytest tests
 .PHONY: test
 
 manylinux:
 	@$(MAKE) -C .. manylinux
 .PHONY: manylinux
 
-wheel: venv
-	@../venv/bin/python setup.py bdist_wheel
+wheel: .venv
+	@../.venv/bin/python setup.py bdist_wheel
 .PHONY: wheel
 
-sdist: venv
-	@../venv/bin/python setup.py sdist --format=zip
+sdist: .venv
+	@../.venv/bin/python setup.py sdist --format=zip
 .PHONY: sdist
 
 clean:

scripts/docker-manylinux.sh

@@ -13,4 +13,4 @@ docker run \
 # Fix permissions for shared directories
 USER_ID=$(id -u)
 GROUP_ID=$(id -g)
-sudo chown -R ${USER_ID}:${GROUP_ID} cabi/target/ py/venv/ ${HOME}/.cargo
+sudo chown -R ${USER_ID}:${GROUP_ID} cabi/target/ ${HOME}/.cargo