Skip to content

remove redundant commands from Makefile #170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
csernazs merged 5 commits into from
Sep 3, 2022
Merged

remove redundant commands from Makefile #170

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
8d72f43
remove redundant commands from Makefile
beliaev-maksim Sep 3, 2022
d0d9c4b
Merge branch 'master' into clean_make
beliaev-maksim Sep 3, 2022
f8e5943
make target dependent on poetry.lock and pyproject.toml
beliaev-maksim Sep 3, 2022
9f54082
comment
beliaev-maksim Sep 3, 2022
a91278b
comment
beliaev-maksim Sep 3, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 9 additions & 14 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,30 +1,25 @@

export POETRY_VIRTUALENVS_IN_PROJECT=true
EXTRAS ?= develop
SPHINXOPTS ?= -n

.venv/.st-venv-completed:
# do poetry install only in case if poetry.lock or pyproject.toml where updated and
# we require a rebuilt.
.venv/.st-venv-completed: poetry.lock pyproject.toml
poetry install --verbose --with $(EXTRAS)
touch .venv/.st-venv-completed

.PHONY: venv
venv: .venv/.st-venv-completed

.PHONY: dev
dev: venv
dev: .venv/.st-venv-completed

.PHONY: precommit
precommit: venv
precommit: dev
poetry run pre-commit run -a

.PHONY: mypy
mypy: venv
mypy: dev
.venv/bin/mypy

.PHONY: autoformat
autoformat: dev
.venv/bin/autopep8 --in-place --recursive pytest_httpserver tests

.PHONY: mrproper
mrproper: clean
rm -rf dist
Expand All @@ -34,7 +29,7 @@ clean: cov-clean doc-clean
rm -rf .venv *.egg-info build .eggs __pycache__ */__pycache__ .tox

.PHONY: test
test: venv
test: dev
.venv/bin/pytest tests -s -vv --release
.venv/bin/pytest tests -s -vv --ssl

Expand All @@ -53,15 +48,15 @@ cov-clean:
rm -rf htmlcov coverage.xml .coverage

.PHONY: doc
doc: .venv
doc: dev
.venv/bin/sphinx-build -M html doc doc/_build $(SPHINXOPTS) $(O)

.PHONY: doc-clean
doc-clean:
rm -rf doc/_build

.PHONY: changes
changes: venv
changes: dev
.venv/bin/reno report --output CHANGES.rst --no-show-source
poetry run pre-commit run --files CHANGES.rst || true
poetry run pre-commit run --files CHANGES.rst