Merge pull request #37 from MarcoGorelli/venv

use .venv, not venv

.github/workflows/pytest.yml

@@ -42,7 +42,7 @@ jobs:
       - name: Set up Rust
         run: rustup show
       - uses: mozilla-actions/[email protected]
-      - run: make venv
+      - run: make .venv
       - run: make pre-commit
       - run: make install
       - run: make test

Makefile

@@ -1,29 +1,29 @@
 
 SHELL=/bin/bash
 
-venv:  ## Set up virtual environment
-	python3 -m venv venv
-	venv/bin/pip install -r requirements.txt
-	venv/bin/pip install -r requirements-dev.txt
+.venv:  ## Set up virtual environment
+	python3 -m venv .venv
+	.venv/bin/pip install -r requirements.txt
+	.venv/bin/pip install -r requirements-dev.txt
 
-install: venv
+install: .venv
 	unset CONDA_PREFIX && \
-	source venv/bin/activate && maturin develop -m Cargo.toml
+	source .venv/bin/activate && maturin develop -m Cargo.toml
 
-install-release: venv
+install-release: .venv
 	unset CONDA_PREFIX && \
-	source venv/bin/activate && maturin develop --release -m Cargo.toml
+	source .venv/bin/activate && maturin develop --release -m Cargo.toml
 
-pre-commit: venv
+pre-commit: .venv
 	cargo fmt --all --manifest-path Cargo.toml && cargo clippy --all-features --manifest-path Cargo.toml
-	venv/bin/python -m ruff format minimal_plugin test_plugin.py
-	venv/bin/python -m ruff check minimal_plugin test_plugin.py
+	.venv/bin/python -m ruff format minimal_plugin test_plugin.py
+	.venv/bin/python -m ruff check minimal_plugin test_plugin.py
 
 run: install
-	source venv/bin/activate && python run.py
+	source .venv/bin/activate && python run.py
 
 run-release: install-release
-	source venv/bin/activate && python run.py
+	source .venv/bin/activate && python run.py
 
-test: venv
-	source venv/bin/activate && pytest test_plugin.py
+test: .venv
+	source .venv/bin/activate && pytest test_plugin.py