Open-Source-Catalog icon indicating copy to clipboard operation
Open-Source-Catalog copied to clipboard

File opened without the with statement, critical anti-pattern

Open philipjonsen opened this issue 2 years ago • 2 comments

DESCRIPTION: Opening a file using with statement is preferred as function open implements the context manager protocol that releases the resource when it is outside of the with block. Not doing so requires you to manually release the resource.

BAD PRACTICE: f = open('/tmp/.deepsource.toml', 'w') f.write("config file.")

No f.close() statement: file may remain unaccessible

PREFERRED: with open('/tmp/.deepsource.toml', 'w') as f: f.write("config file.")

Use the with statement to open a file in these places: /Open-Source-Catalog/blob/master/check-catalog.py#L4-L4 Open-Source-Catalog/blob/master/check-catalog.py#L8-L8

philipjonsen avatar Sep 22 '23 17:09 philipjonsen

Hey i want solve this issue, can i ?

Borsesumit02 avatar Aug 29 '24 12:08 Borsesumit02

Hi! I’d like to work on this issue. Please assign it to me if it’s available. Thank you!

Krishprajapati15 avatar May 24 '25 14:05 Krishprajapati15