Skip to content

Commit

Permalink
Merge pull request #1 from mujin/fixTypoHttpClientResponseHandler
Browse files Browse the repository at this point in the history
fix typo.
  • Loading branch information
ziyan authored Sep 27, 2023
2 parents aced77a + e953a20 commit d055326
Show file tree
Hide file tree
Showing 2 changed files with 31 additions and 4 deletions.
31 changes: 29 additions & 2 deletions .github/workflows/python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,38 @@ on:
branches: [ master ]

jobs:
build:

build_python2:
runs-on: ubuntu-20.04
container:
image: python:2.7.18-buster
strategy:
matrix:
python-version: [2.7, 3.9]
python-version: [2.7]

steps:
- uses: actions/checkout@v2

- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install flake8 pytest
pip install .
- name: Lint with flake8
run: |
flake8 --isolated --show-source --ignore=C901,E128,E201,E202,E203,E221,E225,E226,E227,E228,E231,E241,E251,E261,E262,E265,E271,E301,E302,E303,E305,E402,E501,W291,W293,W391 python
- name: Test with pytest
run: |
pytest || true
build_python3:

runs-on: ubuntu-20.04
strategy:
matrix:
python-version: [3.9]

steps:
- uses: actions/checkout@v2
Expand All @@ -38,3 +64,4 @@ jobs:
- name: Test with pytest
run: |
pytest || true
4 changes: 2 additions & 2 deletions python/mujinasync/asynchttp.py
Original file line number Diff line number Diff line change
Expand Up @@ -199,8 +199,8 @@ def _HandleHttpReceiveOnce(self, connection):
self._HandleHttpResponse(connection, response)
return True # handled one request, try next one

def _HandleHttpResponse(self, connection, request):
return self._CallApi('HandleHttpResponse', request=request, connection=connection, client=self)
def _HandleHttpResponse(self, connection, response):
return self._CallApi('HandleHttpResponse', response=response, connection=connection, client=self)

def _TryReceiveHttpResponse(self, connection):
bufferData = connection.receiveBuffer.readView.tobytes()
Expand Down

0 comments on commit d055326

Please sign in to comment.