hpcugent / vsc-install

Shared setuptools functions and classes for python libraries developed by UGent's HPC group
Other
0 stars 27 forks source link

add consider-using-with to prospector tests #212

Closed wdpypere closed 1 year ago

wdpypere commented 1 year ago

I do regard this as good practice, as the close() is often forgotten (one example even here in vsc-install).

you can read more about it here: https://pylint.pycqa.org/en/latest/user_guide/messages/refactor/consider-using-with.html

But I don't know how many people will be annoyed by this.

wdpypere commented 1 year ago

I ran into this when doing CTRL+c at an extremely unlucky moment :smile: