mik-laj commented on a change in pull request #6957: [AIRFLOW-6396] Use
tempfile.TemporaryDirectory instead of custom one
URL: https://github.com/apache/airflow/pull/6957#discussion_r361879362
##########
File path: UPDATING.md
##########
@@ -41,6 +41,10 @@ assists users migrating to a new version.
## Airflow Master
+### Remove airflow.utils.file.TemporaryDirectory
+
+Removing this because the same functionality is provided by
`tempfile.TemporaryDirectory`.
Review comment:
Can you add more information here?
In my opinion, a good note should contain the following information:
[ ] Previous behaviors
[ ] New behaviors
[ ] If possible, a simple example of how to make a change
[ ] If possible, the benefit for the user after migration e.g. "we want to
make these changes to unify class names."
[ ] If possible, the reason for the change e.g. reference for AIP-21
Here is a similar example that includes most if not all of the requirements:
https://github.com/apache/airflow/blob/master/UPDATING.md#removal-of-redirect_stdout-redirect_stderr
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
