Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @Kircheneer and @sfraint)

a discussion (no related file):
This looks cool, thanks for the contribution. A few notes:

1. We didn't quite have our new GitHub-based CI setup to run on PRs, fixed now. I merged the updated config in, and it found a bunch of style issues. Most were fixed automatically (pre-commit run --all-files), please take a pass.

2. I'd recommend a simpler / less invasive approach that doesn't require quite so much boilerplate. TL;DR:

   • introduce a (private-ish) class that is a SingleFileAnonymizer. That class will be the single source of truth for all these settings, and also only do the logic ones.
   • give it an anonymize_io(in_io, out_io) function that does the work
   • make the existing function create the class once and then recursively call anonymize_io using file IO objects
   • make your new function create the class once and call it with StringIO (or BytesIO, I forget which)

Seems like this approach would be simpler because we'd eliminate the need for this new kwargs dictionary as well as provide reusability.

Reviewed 2 of 3 files at r2, 1 of 1 files at r3, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @Kircheneer and @sfraint)

tests/unit/test_anonymize_files.py line 9 at r3 (raw file):

from netconan.anonymize_files import FileAnonymizer, anonymize_files

Do you know how this got added? Wasn't triggered by pre-commit run --all-files for me. Please revert if so.

Reviewed 1 of 1 files at r4, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @Kircheneer and @sfraint)

Reviewed 1 of 1 files at r5, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @Kircheneer and @sfraint)