Add os.walk wrapper - Githubissues

Tinche / aiofiles

File support for asyncio

Apache License 2.0

2.88k stars 153 forks source link

Open stankudrow opened 2 weeks ago

stankudrow commented 2 weeks ago

Addresses the issue #160 .

Features:

Now the wrap function is placed in the base.py module as a basic function of sorts -> importing the wrap function from the os.py or ospath.py are not expected to be down
the wrapped functions got arranged alphabetically and form sections split by new lines
added a test on the wrapped os.walk coroutine

stankudrow commented 1 week ago

Blocked by the PR #192 .

Update: not any more.

stankudrow commented 1 week ago

stankudrow commented 1 week ago

@Tinche , hello. Rebased and ready for review, all checks have been passed.

Update: I found the issue #167 on an "invented-here" solution, what are your suggestions?

stankudrow commented 22 hours ago

@Tinche , hello, shall we review this PR, please?