Abstract pexpect.SpawnBase has no send/sendline/write/writelines stubs
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 5.1k
- Forks
- 2.1k
- Avg merge
- 1d 19h
- Merged PRs (30d)
- 82
Description
The actual implementation of pexpect.SpawnBase does not contain these functions but it is an abstract class and all children have these write methods.
Since almost all interesting pexpect code does some form of writing it is impractical to use SpawnBase as an annotation. The only viable solution seem to be annotating with PopenSpawn or maybe an union if you actually want to be generic?
Perhaps it would be acceptable to add these stubs to SpawnBase despite what is written in the implementation.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the SpawnBase stub in typeshed and compare it with the child stubs that define send, sendline, write, and writelines. Check the relevant typing tests or run the repository's type-checking tests to confirm that SpawnBase annotations expose these methods without breaking existing implementations.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 62/100