Skip to content

Conversation

@altendky
Copy link
Collaborator

coroutine is no longer available via the asyncio module as of Python 3.11.

$ venv311/bin/python -c 'import memory_profiler'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/altendky//venv311/lib/python3.11/site-packages/memory_profiler.py", line 10, in <module>
    from asyncio import coroutine, iscoroutinefunction
ImportError: cannot import name 'coroutine' from 'asyncio' (/home/altendky/.pyenv/versions/3.11.0rc2/lib/python3.11/asyncio/__init__.py)

`coroutine` is no longer available via the `asyncio` module as of Python 3.11.

```python-traceback
$ venv311/bin/python -c 'import memory_profiler'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/altendky//venv311/lib/python3.11/site-packages/memory_profiler.py", line 10, in <module>
    from asyncio import coroutine, iscoroutinefunction
ImportError: cannot import name 'coroutine' from 'asyncio' (/home/altendky/.pyenv/versions/3.11.0rc2/lib/python3.11/asyncio/__init__.py)
```
@altendky
Copy link
Collaborator Author

Looks like the failure is due to codespell processing the .git directory which should probably be excluded.

@fabianp
Copy link
Collaborator

fabianp commented Oct 21, 2022

could you submit a fix for that?

@altendky
Copy link
Collaborator Author

Trying at #369.

@altendky
Copy link
Collaborator Author

kick ci

@altendky altendky closed this Oct 21, 2022
@altendky altendky reopened this Oct 21, 2022
@altendky
Copy link
Collaborator Author

Thoughts on this now? I figured it would be helpful to test and show this being broken so I added a matrix across the Python versions but had a question at #370 (comment).

@fabianp fabianp merged commit 4b81aee into pythonprofilers:master Nov 1, 2022
@fabianp
Copy link
Collaborator

fabianp commented Nov 1, 2022

great, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants