chore: add support for PTH linter in ruff

Author: remyleone

scaleway-async/pyproject.toml

@@ -46,6 +46,7 @@ select = [
     "FBT", # https://docs.astral.sh/ruff/rules/#flake8-boolean-trap-fbt
     "FIX", # https://docs.astral.sh/ruff/rules/#flake8-fixme-fix
     "FURB", # https://docs.astral.sh/ruff/rules/#refurb-furb
+    "PTH", # https://docs.astral.sh/ruff/rules/#flake8-use-pathlib-pth
     "T10", # https://docs.astral.sh/ruff/rules/#flake8-debugger-t10
     "YTT", # https://docs.astral.sh/ruff/rules/#flake8-2020-ytt
 ]

scaleway-core/pyproject.toml

@@ -46,6 +46,7 @@ select = [
     "ERA", # https://docs.astral.sh/ruff/rules/#eradicate-era
     "FIX", # https://docs.astral.sh/ruff/rules/#flake8-fixme-fix
     "FURB", # https://docs.astral.sh/ruff/rules/#refurb-furb
+    "PTH", # https://docs.astral.sh/ruff/rules/#flake8-use-pathlib-pth
     "T10", # https://docs.astral.sh/ruff/rules/#flake8-debugger-t10
     "YTT", # https://docs.astral.sh/ruff/rules/#flake8-2020-ytt
 ]

scaleway-core/scaleway_core/profile/profile.py

@@ -4,6 +4,7 @@
 import logging
 import os
 from dataclasses import dataclass
+from pathlib import Path
 from typing import Optional, Type, TypeVar
 
 import yaml
@@ -110,23 +111,23 @@ def from_env(cls: Type[ProfileSelf], force_none: bool = False) -> ProfileSelf:
         return profile
 
     @classmethod
-    def get_default_config_directory(cls) -> str:
+    def get_default_config_directory(cls) -> Path:
         xdg_config_path = os.environ.get("XDG_CONFIG_HOME")
         if xdg_config_path is not None and xdg_config_path != "":
-            return os.path.join(xdg_config_path, "scw")
+            return Path(xdg_config_path) / "scw"
 
-        return os.path.join(os.path.expanduser("~"), ".config", "scw")
+        return Path.expanduser("~") / ".config" / "scw"
 
     @classmethod
-    def get_default_config_file_path(cls, filepath: Optional[str] = None) -> str:
+    def get_default_config_file_path(cls, filepath: Optional[str] = None) -> Path:
         if filepath is not None:
-            return filepath
+            return Path(filepath)
 
         filepath = os.environ.get(ENV_KEY_SCW_CONFIG_PATH)
         if filepath is not None and filepath != "":
-            return filepath
+            return Path(filepath)
 
-        return os.path.join(Profile.get_default_config_directory(), "config.yaml")
+        return Profile.get_default_config_directory() / "config.yaml"
 
     @classmethod
     def from_config_file(
@@ -137,7 +138,7 @@ def from_config_file(
     ) -> ProfileSelf:
         filepath = cls.get_default_config_file_path(filepath)
 
-        with open(filepath, "r") as f:
+        with Path(filepath).open("r") as f:
             config = yaml.safe_load(f)
 
             if not isinstance(config, dict):

scaleway-core/tests/test_profile_file.py

@@ -4,6 +4,7 @@
 import tempfile
 import unittest
 import uuid
+from pathlib import Path
 from unittest import mock
 
 import utils
@@ -56,7 +57,7 @@ def setUp(self) -> None:
         fp.close()
 
     def tearDown(self) -> None:
-        os.unlink(self.profile_file_name)
+        Path.unlink(self.profile_file_name)
 
     def test_load_profile_from_config_file(self) -> None:
         profile = Profile.from_config_file(self.profile_file_name)

scaleway/pyproject.toml

@@ -47,6 +47,7 @@ select = [
     "FBT", # https://docs.astral.sh/ruff/rules/#flake8-boolean-trap-fbt
     "FIX", # https://docs.astral.sh/ruff/rules/#flake8-fixme-fix
     "FURB", # https://docs.astral.sh/ruff/rules/#refurb-furb
+    "PTH", # https://docs.astral.sh/ruff/rules/#flake8-use-pathlib-pth
     "T10", # https://docs.astral.sh/ruff/rules/#flake8-debugger-t10
     "YTT", # https://docs.astral.sh/ruff/rules/#flake8-2020-ytt
 ]