Skip to content

Commit

Permalink
Fix PR-comment GitHub Action by using our forked version. (#36)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
daniel-k authored Jan 24, 2024
1 parent e79438d commit caa69aa
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/pull_request.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down
2 changes: 0 additions & 2 deletions src/enlyze/client.py
Original file line number Diff line number Diff line change
Expand Up @@ -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|
Expand Down Expand Up @@ -138,7 +137,6 @@ def get_variables(
"""Retrieve all variables of an :ref:`appliance <appliance>`.
:param appliance: The appliance for which to get all variables.
:type appliance: :class:`~enlyze.models.Appliance`
:raises: |token-error|
Expand Down

0 comments on commit caa69aa

Please sign in to comment.