Snapchat / Snapchat/GiGL

Use context manager for __file_loader.load_to_temp_file

Open
#285 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
85
Forks
19
Avg merge
4d 10h
Merged PRs (30d)
6

Description

```
with self.__file_loader.load_to_temp_file(file_uri_src=uri, delete=False) as tfh:
with open(thf, 'w') as f:
...
```
without the `tfh.close()` call?

_Originally posted by @kmontemayor2-sc in https://github.com/Snapchat/GiGL/pull/279#discussion_r2285742099_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.