From caa69aacf1d53ec32a86efb8368775f33a6f9b6f Mon Sep 17 00:00:00 2001 From: Daniel Krebs Date: Wed, 24 Jan 2024 17:30:29 +0100 Subject: [PATCH] Fix PR-comment GitHub Action by using our forked version. (#36) I had to remove a problematic type annotation in the docstring of `EnlyzeClient.get_appliances()` as well, because that has led to a failure building the docs. For consistency, I've also removed a similar annotation in the same class. Both annotation are not strictly necessary, so this shouldn't be a big deal. --- .github/workflows/pull_request.yaml | 2 +- src/enlyze/client.py | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/.github/workflows/pull_request.yaml b/.github/workflows/pull_request.yaml index 7815ade..3824cb9 100644 --- a/.github/workflows/pull_request.yaml +++ b/.github/workflows/pull_request.yaml @@ -75,7 +75,7 @@ jobs: - name: Report test coverage if: ${{ matrix.coverage-report == true }} - uses: rhs/pr-comment@1.0.4 + uses: enlyze/pr-comment@v1.0.5 env: GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} with: diff --git a/src/enlyze/client.py b/src/enlyze/client.py index ec4aa11..1f07224 100644 --- a/src/enlyze/client.py +++ b/src/enlyze/client.py @@ -96,7 +96,6 @@ def get_appliances( :param site: Only get appliances of this site. Gets all appliances of the organization if None. - :type site: :class:`~enlyze.models.Site` or None :raises: |token-error| @@ -138,7 +137,6 @@ def get_variables( """Retrieve all variables of an :ref:`appliance `. :param appliance: The appliance for which to get all variables. - :type appliance: :class:`~enlyze.models.Appliance` :raises: |token-error|