chore: remove the dev setup script

.github/workflows/ci.yml

@@ -33,19 +33,15 @@ jobs:
           path: ~/.cache/pre-commit
           key: pre-commit-3|${{ env.pythonLocation }}|${{ hashFiles('.pre-commit-config.yaml') }}
 
-      - name: Install Dependencies
-        run: source ./setup_dev_env.sh
-
       - name: Run pre-commit checks
         run: |
-          source .venv/bin/activate
-          pre-commit run --all-files --show-diff-on-failure --color always
+          uv run pre-commit run --all-files --show-diff-on-failure --color always
 
       - name: Run ruff formatter
-        run: uvx ruff format --check
+        run: uv run ruff format --check
 
       - name: Run ruff linter
-        run: uvx ruff check
+        run: uv run ruff check
 
       - name: Run mypy
         run: uv run mypy .
@@ -79,18 +75,15 @@ jobs:
 
       - name: Check licenses
         run: |
-          source .venv/bin/activate
-          ./check_licenses.sh
+          uv run ./check_licenses.sh
 
       - name: Check documentation builds correctly
         run: |
-          source .venv/bin/activate
-          mkdocs build --strict
+          uv run mkdocs build --strict
 
       - name: Generate pip freeze
         run: |
-          source .venv/bin/activate
-          pip freeze > requirements-freeze.txt
+          uv pip freeze > requirements-freeze.txt
 
       - name: Publish Artefacts
         uses: actions/upload-artifact@v3
@@ -167,16 +160,12 @@ jobs:
           path: ~/nltk_data
           key: nltk-${{ runner.os }}
 
-      - name: Install Dependencies
-        run: source ./setup_dev_env.sh
-
       - name: Run Tests With Coverage
         run: |
           # run with coverage to not execute tests twice
-          source .venv/bin/activate
-          coverage run -m pytest -v -p no:warnings --junitxml=report.xml
-          coverage report
-          coverage xml
+          uv run coverage run -m pytest -v -p no:warnings --junitxml=report.xml
+          uv run coverage report
+          uv run coverage xml
 
       - name: Test Report
         uses: mikepenz/action-junit-report@v4

.github/workflows/documentation.yaml

@@ -33,13 +33,9 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-pip-
 
-      - name: Install Dependencies
-        shell: bash
-        run: source ./setup_dev_env.sh
-
       - name: Deploy docs
         shell: bash
-        run: ./.github/scripts/deploy_docs.sh
+        run: uv run ./.github/scripts/deploy_docs.sh
         env:
           GCP_KEY: ${{ secrets.GCP_KEY }}
 

CONTRIBUTING.md

@@ -2,25 +2,22 @@
 
 ## Build from source
 
-To build and run Ragbits from the source code:
+Dependencies needed to build and run Ragbits from the source code:
 
-1. Requirements: [**uv**](https://docs.astral.sh/uv/getting-started/installation/) & [**python**](https://docs.astral.sh/uv/guides/install-python/) 3.10 or higher
-2. Install dependencies and run venv in editable mode:
+1. [**uv**](https://docs.astral.sh/uv/getting-started/installation/)
+2. [**python**](https://docs.astral.sh/uv/guides/install-python/) 3.10
 
-```bash
-$ source ./setup_dev_env.sh
-```
 
 ## Linting and formatting
 We use `ruff` for linting and formatting our code. To format your code, run:
 
 ```bash
-$ uvx ruff format
+$ uv run ruff format
 ```
 
 To lint the code, run:
 ```bash
-$ uvx ruff check --fix
+$ uv run ruff check --fix
 ```
 
 ## Type checking

check_licenses.sh

@@ -1,8 +1,6 @@
 #!/bin/bash
 set -e
 
-source .venv/bin/activate
-
 uv run pip-licenses --from=mixed  --ignore-packages `cat .libraries-whitelist.txt`> licenses.txt
 cat licenses.txt
 

packages/ragbits-document-search/tests/unit/test_document_search.py

@@ -290,9 +290,9 @@ async def test_document_search_ingest_from_uri_with_wildcard(
         results = await document_search.search(search_query)
 
         # Check that we have the expected number of results
-        assert len(results) == len(expected_contents), (
-            f"Expected {len(expected_contents)} result(s) but got {len(results)}"
-        )
+        assert len(results) == len(
+            expected_contents
+        ), f"Expected {len(expected_contents)} result(s) but got {len(results)}"
 
         # Verify each result is a TextElement
         assert all(isinstance(result, TextElement) for result in results)