Testgres has two lines with strange and "useless" code: https://github.com/postgrespro/testgres/blob/9da89812c04f987e6833e3bbaf43ce922d34105a/testgres/node.py#L2387 https://github.com/postgrespro/testgres/blob/9da89812c04f987e6833e3bbaf43ce922d34105a/testgres/backup.py#L168 These things (should_rm_dirs and _should_rm_dirs) are not used and not processed anywhere. We may delete they without any problems. Am I right?