Skip to content

fix: use context managers for opened files

David Flor requested to merge xflord/fix_file_context into main

Description

fix: use context managers for opened files

  • ruff extended their rules to also check for wider range of libraries, one which we use
  • added contexts to pass the rules

How to test

Not sure, I would appreciate if people that did the refactor of generic_sender checked it out and tested it if they already have the environment ready

Author's checklist

  • I have followed the contribution guidelines
  • This MR has been tested or does not change functionality
  • I have added relevant merge request dependencies (if this MR has any)
  • I have added the correct labels
  • I have assigned reviewers (if any are relevant)
  • I have edited the documentation (if the changes require it) or I have noted the need for the change if I do not have access to the documentation
  • I have marked all introduced BREAKING CHANGES or necessary DEPLOYMENT NOTES in the commit message(s)

Reviewer's checklist

  • This MR has been tested or does not change functionality
  • This MR has correct commit message format

Other information

N/A

Related issues

N/A

Edited by David Flor

Merge request reports