Skip to content

Conversation

@echoix
Copy link
Member

@echoix echoix commented Nov 9, 2025

PTH102 os.mkdir() should be replaced by Path.mkdir()

PTH103 os.makedirs() should be replaced by Path.mkdir(parents=True)

Ruff rules: https://docs.astral.sh/ruff/rules/os-mkdir/ https://docs.astral.sh/ruff/rules/os-makedirs/

Try merging #6603 before, there might be some that were already fixed

…()` with pathlib

PTH102 [*] `os.mkdir()` should be replaced by `Path.mkdir()`
PTH103 [*] `os.makedirs()` should be replaced by `Path.mkdir(parents=True)`

Ruff rules: https://docs.astral.sh/ruff/rules/os-mkdir/
https://docs.astral.sh/ruff/rules/os-makedirs/
@echoix echoix requested a review from ninsbl November 9, 2025 16:37
@github-actions github-actions bot added GUI wxGUI related vector Related to vector data processing raster Related to raster data processing Python Related code is in Python libraries module docs general imagery tests Related to Test Suite labels Nov 9, 2025
Copy link
Member

@ninsbl ninsbl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one suggestion, feel free to disregard.

@echoix echoix merged commit 59f1608 into OSGeo:main Nov 11, 2025
29 checks passed
@echoix echoix deleted the ruff-PTH102-PTH103 branch November 11, 2025 20:17
@github-actions github-actions bot added this to the 8.5.0 milestone Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs general GUI wxGUI related imagery libraries module Python Related code is in Python raster Related to raster data processing tests Related to Test Suite vector Related to vector data processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants