From 9c1ecfe4d479e06d480d63fb05f902033d30bc74 Mon Sep 17 00:00:00 2001 From: Quinn Milionis <35316732+quinnmil@users.noreply.github.com> Date: Tue, 27 Aug 2024 09:47:04 -0700 Subject: [PATCH] Build fixes and docs link (#793) * fix pre-commit check-manifest missing dependencies * update docs url * add missing setuptool dependency for github workflow * provide unique basemane for drf router viewsets --- .github/workflows/main.yml | 4 ++-- .github/workflows/release.yml | 2 +- .pre-commit-config.yaml | 1 + README.md | 2 +- tests/integration/django_app.py | 4 ++-- 5 files changed, 7 insertions(+), 6 deletions(-) diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index e1b0bb66..b1bd4b6d 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -55,7 +55,7 @@ jobs: with: python-version: ${{ matrix.python-version }} - name: Upgrade packaging tools - run: python -m pip install --upgrade pip setuptools virtualenv + run: python -m pip install --upgrade pip setuptools importlib_metadata virtualenv - name: Install dependencies run: python -m pip install --upgrade tox - name: Run tox targets for ${{ matrix.python-version }} @@ -104,7 +104,7 @@ jobs: with: python-version: ${{ matrix.python-version }} - name: Upgrade packaging tools - run: python -m pip install --upgrade pip setuptools virtualenv + run: python -m pip install --upgrade pip setuptools importlib_metadata virtualenv - name: Install dependencies run: python -m pip install --upgrade tox - name: Run tox targets for ${{ matrix.python-version }} diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 738fb77b..21ad6e6f 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -91,7 +91,7 @@ jobs: python-version: ${{ matrix.python-version }} - name: Install packaging tools run: | - python -m pip install --upgrade pip setuptools wheel + python -m pip install --upgrade pip setuptools importlib_metadata wheel - name: Build Python pure Python wheel env: diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 9f4e2cb7..a62853d7 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -42,3 +42,4 @@ repos: hooks: - id: check-manifest args: [--no-build-isolation] + additional_dependencies: [setuptools, wheel, setuptools-scm] diff --git a/README.md b/README.md index fcd87d0d..d4f0480d 100644 --- a/README.md +++ b/README.md @@ -35,7 +35,7 @@ To use Scout, you'll need to For full installation instructions, including information on configuring Scout via environment variables and troubleshooting, see our -[Python docs](https://docs.scoutapm.com/#python-agent). +[Python docs](https://scoutapm.com/docs/python). ## Support diff --git a/tests/integration/django_app.py b/tests/integration/django_app.py index 7c27d3eb..4b68bf18 100644 --- a/tests/integration/django_app.py +++ b/tests/integration/django_app.py @@ -193,8 +193,8 @@ def get_queryset(self, *args, **kwargs): raise ValueError("BØØM!") router = routers.SimpleRouter() - router.register(r"users", UserViewSet) - router.register(r"crash", ErrorViewSet) + router.register(r"users", UserViewSet, basename="user") + router.register(r"crash", ErrorViewSet, basename="error") return router