Author nyuszika7h
Recipients Anthony Sottile, georg.brandl, maurosr, ncoghlan, nyuszika7h, pitrou, r.david.murray, serhiy.storchaka
Date 2020-12-06.14:10:12
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1607263813.35.0.39023302923.issue25024@roundup.psfhosted.org>
In-reply-to
Content
Sorry for reviving a 9 months old issue, but IMO there was no good reason to reject this especially when a patch was provided. Even if the context manager can be replaced with 3 lines of code, I still don't consider that very user-friendly.

My use case would be passing `delete=False` temporarily while debugging my script, it would be much simpler than using a whole different hacky method when the end goal is to change it back to `delete=True` once it is completed anyway.

What issues exactly does the addition of a simple option cause? I don't think something as trivial as this causes a maintenance burden, and you can't call it feature creep either when TemporaryDirectory doesn't have *any* other optional keyword arguments.
History
Date User Action Args
2020-12-06 14:10:13nyuszika7hsetrecipients: + nyuszika7h, georg.brandl, ncoghlan, pitrou, r.david.murray, serhiy.storchaka, maurosr, Anthony Sottile
2020-12-06 14:10:13nyuszika7hsetmessageid: <1607263813.35.0.39023302923.issue25024@roundup.psfhosted.org>
2020-12-06 14:10:13nyuszika7hlinkissue25024 messages
2020-12-06 14:10:12nyuszika7hcreate