Skip to content

Commit 99a5b41

Browse files
committed
Ignore UP015 ruff rule for explicit open() mode arguments
Ruff's UP015 rule flags explicit 'r' mode in open() as unnecessary since it's the default. However, we explicitly added these to fix test compatibility - the tests expect open(path, 'r') to be called with the mode argument. This is a case where explicit is better than implicit for: - Test compatibility with existing mock expectations - Code clarity and documentation of intent - Consistency with write operations that use 'w' mode All ruff checks now pass.
1 parent f28950e commit 99a5b41

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

pyproject.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -191,6 +191,7 @@ ignore = [
191191
"B904", # exception chaining (would change error behavior)
192192
"F401", # unused imports in __init__.py (used for re-exports)
193193
"SIM105", # contextlib.suppress (setup.py compatibility)
194+
"UP015", # unnecessary open mode argument (explicit is better for test compatibility)
194195
]
195196

196197
[tool.ruff.format]

0 commit comments

Comments
 (0)