This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author aeros
Recipients aeros, docs@python, jftuga, python-dev, rhettinger, serhiy.storchaka
Date 2020-04-04.01:16:45
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1585963006.36.0.073094413487.issue40160@roundup.psfhosted.org>
In-reply-to
Content
Serhiy Storchaka wrote:
> I do not think there is clearer example of topdown=False than recursive remove.
>
> If you think that this example is destructive, consider how destructive is any possible example for shutil.rmtree()!

I concur with Serhiy. If the example is changed to just print out the file and directory path, as the PR proposes to do, it seems to defeat the purpose of using `topdown=False` (and the code example) in the first place.

If anything, I would suggest adding succinct comments or a note that very briefly explains how one could see a visual demonstration of what it does without removing any files or directories. For example: ``print(f"os.remove({os.path.join(root, name)})")``.
History
Date User Action Args
2020-04-04 01:16:46aerossetrecipients: + aeros, rhettinger, docs@python, python-dev, serhiy.storchaka, jftuga
2020-04-04 01:16:46aerossetmessageid: <1585963006.36.0.073094413487.issue40160@roundup.psfhosted.org>
2020-04-04 01:16:46aeroslinkissue40160 messages
2020-04-04 01:16:45aeroscreate