Skip to content

Commit

Permalink
chore: install target to use editable install in GH tests
Browse files Browse the repository at this point in the history
  • Loading branch information
jjaakola-aiven committed Sep 25, 2024
1 parent e80335f commit 835a5da
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ jobs:
cache: pip
python-version: ${{ matrix.python-version }}

- run: make install
- run: make install-dev
- run: make unit-tests
env:
COVERAGE_FILE: ".coverage.${{ matrix.python-version }}"
Expand Down
19 changes: 16 additions & 3 deletions GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -42,16 +42,29 @@ venv/.make:

.PHONY: install
install: venv/.deps
venv/.deps: requirements/requirements-dev.txt requirements/requirements.txt | venv/.make
venv/.deps: venv/.make
set +x
source ./bin/get-java
source ./bin/get-protoc
source ./bin/get-snappy
set -x
$(PIP) install --use-pep517 . .[dev]
$(PIP) install --use-pep517 .
$(PIP) check
touch '$(@)'

.PHONY: install-dev
install-dev: venv/.deps-dev
venv/.deps-dev: venv/.make
set +x
source ./bin/get-java
source ./bin/get-protoc
source ./bin/get-snappy
set -x
$(PIP) install -e .[dev]
$(PIP) check
touch '$(@)'


.PHONY: test
tests: unit-tests integration-tests

Expand Down Expand Up @@ -82,7 +95,7 @@ cleanest: cleaner
rm -fr '$(VENV_DIR)'

.PHONY: requirements
requirements: export CUSTOM_COMPILE_COMMAND='make requirements'
requirements:
requirements:
$(PIP) install --upgrade pip setuptools pip-tools
$(PIP) install .[dev,typing]
Expand Down

0 comments on commit 835a5da

Please sign in to comment.