nk-python-style 0.1.0__tar.gz → 0.2.0__tar.gz

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/PKG-INFO

@@ -1,12 +1,11 @@
 Metadata-Version: 2.4
 Name: nk-python-style
-Version: 0.1.0
+Version: 0.2.0
 Summary: Python formatter and linter following the repository's conventions
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 Requires-Dist: ruff<1,>=0.16
-Provides-Extra: comments
-Requires-Dist: lingua-language-detector<3,>=2.2; extra == "comments"
+Requires-Dist: lingua-language-detector<3,>=2.2
 
 # Python : linter et formatter
 
@@ -20,11 +19,7 @@ Depuis ce dépôt :
 python -m pip install -e ./python
 ```
 
-Pour activer la détection de la langue des commentaires, installer l'extra optionnel (Lingua est volumineux) :
-
-```bash
-python -m pip install -e "./python[comments]"
-```
+L'installation inclut Ruff et Lingua. La détection de la langue des commentaires est toujours active.
 
 Depuis un autre projet, une installation non éditable fonctionne aussi : `python -m pip install <chemin-vers-ce-dépôt>/python`. La commande `nk-python` est ensuite disponible dans le même environnement Python. On peut également utiliser `python -m nk_python`.
 
@@ -52,7 +47,7 @@ La sortie de `check` commence par un résumé : fichiers analysés, nombre de wa
 | `PY004` | Cinq paramètres maximum ; `self` ou `cls` est exclu pour les méthodes. |
 | `PY005` | Commentaire de fin de ligne interdit, sauf directive d'outil. |
 | `PY006` | Commentaire dans une fonction interdit, sauf directive d'outil ; utiliser une docstring ou clarifier le code. |
-| `PY007` | Commentaire qui semble non anglais ; actif si l'extra `comments` est installé. Les textes trop courts ou ambigus sont ignorés. |
+| `PY007` | Commentaire qui semble non anglais. Les textes trop courts ou ambigus sont ignorés. |
 | `PY008` | Huit fichiers `.py`/`.pyi` directs maximum dans un dossier parcouru. |
 
 Ruff vérifie en plus les erreurs Python (`E4`, `E7`, `E9`, `F`), l'ordre des imports (`I`), les erreurs fréquentes (`B`), certaines modernisations sûres (`UP`) et la variable créée uniquement pour être retournée (`RET504`). Sa configuration est fournie par l'outil, donc les mêmes règles s'appliquent depuis un autre projet. Le formatter impose une indentation de quatre espaces, des guillemets doubles et des fins de ligne LF. Les règles C++ propres aux accolades, `#include`, modules et initialiseurs désignés ne s'appliquent pas à Python.

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/README.md

@@ -10,11 +10,7 @@ Depuis ce dépôt :
 python -m pip install -e ./python
 ```
 
-Pour activer la détection de la langue des commentaires, installer l'extra optionnel (Lingua est volumineux) :
-
-```bash
-python -m pip install -e "./python[comments]"
-```
+L'installation inclut Ruff et Lingua. La détection de la langue des commentaires est toujours active.
 
 Depuis un autre projet, une installation non éditable fonctionne aussi : `python -m pip install <chemin-vers-ce-dépôt>/python`. La commande `nk-python` est ensuite disponible dans le même environnement Python. On peut également utiliser `python -m nk_python`.
 
@@ -42,7 +38,7 @@ La sortie de `check` commence par un résumé : fichiers analysés, nombre de wa
 | `PY004` | Cinq paramètres maximum ; `self` ou `cls` est exclu pour les méthodes. |
 | `PY005` | Commentaire de fin de ligne interdit, sauf directive d'outil. |
 | `PY006` | Commentaire dans une fonction interdit, sauf directive d'outil ; utiliser une docstring ou clarifier le code. |
-| `PY007` | Commentaire qui semble non anglais ; actif si l'extra `comments` est installé. Les textes trop courts ou ambigus sont ignorés. |
+| `PY007` | Commentaire qui semble non anglais. Les textes trop courts ou ambigus sont ignorés. |
 | `PY008` | Huit fichiers `.py`/`.pyi` directs maximum dans un dossier parcouru. |
 
 Ruff vérifie en plus les erreurs Python (`E4`, `E7`, `E9`, `F`), l'ordre des imports (`I`), les erreurs fréquentes (`B`), certaines modernisations sûres (`UP`) et la variable créée uniquement pour être retournée (`RET504`). Sa configuration est fournie par l'outil, donc les mêmes règles s'appliquent depuis un autre projet. Le formatter impose une indentation de quatre espaces, des guillemets doubles et des fins de ligne LF. Les règles C++ propres aux accolades, `#include`, modules et initialiseurs désignés ne s'appliquent pas à Python.

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/nk_python/cli.py

@@ -7,7 +7,7 @@ import sys
 from pathlib import Path
 
 from .ast_checks import check_source
-from .comment_checks import LANGUAGE_AVAILABLE, check_comments
+from .comment_checks import check_comments
 from .model import Issue
 from .ruff_runner import run_ruff
 from .structure_checks import discover
@@ -70,9 +70,6 @@ def _check(files: list[Path], structure_issues: list[Issue]) -> int:
     print(lint_text, end="")
     for issue in sorted(issues):
         print(issue)
-    if not LANGUAGE_AVAILABLE:
-        print("Comment language check skipped (install nk-python-style[comments] to enable it).")
-
     print("\n=== Formatter: corrections available via 'format' ===")
     print(import_text, end="")
     print(format_text, end="")

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/nk_python/comment_checks.py

@@ -1,19 +1,14 @@
-"""Token-based comment placement and optional language checks."""
+"""Token-based comment placement and language checks."""
 
 import io
 import re
 import tokenize
 from pathlib import Path
 
-from .model import Issue
+from lingua import Language, LanguageDetectorBuilder
 
-try:
-    from lingua import Language, LanguageDetectorBuilder
-except ImportError:
-    Language = None
-    LanguageDetectorBuilder = None
+from .model import Issue
 
-LANGUAGE_AVAILABLE = LanguageDetectorBuilder is not None
 _detector = None
 _DIRECTIVE = re.compile(
     r"^(?:!|coding[:=]|noqa\b|type:\s*ignore\b|pyright:\s*ignore\b|"
@@ -52,8 +47,6 @@ def check_comments(path: Path, source: str, functions: list[tuple[int, int]]) ->
 
 
 def _comment_language(comment: str) -> str | None:
-    if not LANGUAGE_AVAILABLE:
-        return None
     words = [word for word in _WORD.findall(comment) if _natural_word(word)]
     if len(words) < 3 or len(" ".join(words)) < 20:
         return None

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/nk_python/ruff.toml

@@ -6,6 +6,7 @@ extend-exclude = [".git", ".venv*", "venv", "__pycache__", "build", "dist", "nod
 
 [lint]
 select = ["E4", "E7", "E9", "F", "I", "B", "UP", "RET504"]
+ignore = ["B008", "B028", "B905", "F811"]
 
 [format]
 indent-style = "space"

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/nk_python_style.egg-info/PKG-INFO

@@ -1,12 +1,11 @@
 Metadata-Version: 2.4
 Name: nk-python-style
-Version: 0.1.0
+Version: 0.2.0
 Summary: Python formatter and linter following the repository's conventions
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 Requires-Dist: ruff<1,>=0.16
-Provides-Extra: comments
-Requires-Dist: lingua-language-detector<3,>=2.2; extra == "comments"
+Requires-Dist: lingua-language-detector<3,>=2.2
 
 # Python : linter et formatter
 
@@ -20,11 +19,7 @@ Depuis ce dépôt :
 python -m pip install -e ./python
 ```
 
-Pour activer la détection de la langue des commentaires, installer l'extra optionnel (Lingua est volumineux) :
-
-```bash
-python -m pip install -e "./python[comments]"
-```
+L'installation inclut Ruff et Lingua. La détection de la langue des commentaires est toujours active.
 
 Depuis un autre projet, une installation non éditable fonctionne aussi : `python -m pip install <chemin-vers-ce-dépôt>/python`. La commande `nk-python` est ensuite disponible dans le même environnement Python. On peut également utiliser `python -m nk_python`.
 
@@ -52,7 +47,7 @@ La sortie de `check` commence par un résumé : fichiers analysés, nombre de wa
 | `PY004` | Cinq paramètres maximum ; `self` ou `cls` est exclu pour les méthodes. |
 | `PY005` | Commentaire de fin de ligne interdit, sauf directive d'outil. |
 | `PY006` | Commentaire dans une fonction interdit, sauf directive d'outil ; utiliser une docstring ou clarifier le code. |
-| `PY007` | Commentaire qui semble non anglais ; actif si l'extra `comments` est installé. Les textes trop courts ou ambigus sont ignorés. |
+| `PY007` | Commentaire qui semble non anglais. Les textes trop courts ou ambigus sont ignorés. |
 | `PY008` | Huit fichiers `.py`/`.pyi` directs maximum dans un dossier parcouru. |
 
 Ruff vérifie en plus les erreurs Python (`E4`, `E7`, `E9`, `F`), l'ordre des imports (`I`), les erreurs fréquentes (`B`), certaines modernisations sûres (`UP`) et la variable créée uniquement pour être retournée (`RET504`). Sa configuration est fournie par l'outil, donc les mêmes règles s'appliquent depuis un autre projet. Le formatter impose une indentation de quatre espaces, des guillemets doubles et des fins de ligne LF. Les règles C++ propres aux accolades, `#include`, modules et initialiseurs désignés ne s'appliquent pas à Python.

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/nk_python_style.egg-info/SOURCES.txt

@@ -16,5 +16,6 @@ nk_python_style.egg-info/entry_points.txt
 nk_python_style.egg-info/requires.txt
 nk_python_style.egg-info/top_level.txt
 tests/test_cli.py
+tests/test_ruff_config.py
 tests/test_rules.py
 tests/test_summary.py

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/nk_python_style.egg-info/requires.txt

@@ -1,4 +1,2 @@
 ruff<1,>=0.16
-
-[comments]
 lingua-language-detector<3,>=2.2

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/pyproject.toml

@@ -4,14 +4,11 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "nk-python-style"
-version = "0.1.0"
+version = "0.2.0"
 description = "Python formatter and linter following the repository's conventions"
 readme = "README.md"
 requires-python = ">=3.10"
-dependencies = ["ruff>=0.16,<1"]
-
-[project.optional-dependencies]
-comments = ["lingua-language-detector>=2.2,<3"]
+dependencies = ["ruff>=0.16,<1", "lingua-language-detector>=2.2,<3"]
 
 [project.scripts]
 nk-python = "nk_python.cli:main"

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/tests/test_cli.py

@@ -6,6 +6,23 @@ from pathlib import Path
 
 
 class CliTests(unittest.TestCase):
+    def test_check_reports_non_english_comments(self):
+        with tempfile.TemporaryDirectory() as temp:
+            path = Path(temp) / "sample.py"
+            source = "# Cette fonction calcule la valeur finale pour chaque entrée.\nvalue = 1\n"
+            path.write_text(source, encoding="utf-8")
+            checked = subprocess.run(
+                [sys.executable, "-m", "nk_python", "check", str(path)],
+                capture_output=True,
+                text=True,
+                check=False,
+            )
+            self.assertEqual(checked.returncode, 1, checked.stderr + checked.stdout)
+            self.assertIn("PY007", checked.stdout)
+            self.assertIn("Linter: 1 warnings", checked.stdout)
+            self.assertNotIn("All checks passed!", checked.stdout)
+            self.assertEqual(path.read_text(encoding="utf-8"), source)
+
     def test_check_separates_linter_diagnostics_and_formatter_corrections(self):
         with tempfile.TemporaryDirectory() as temp:
             path = Path(temp) / "sample.py"

nk_python_style-0.2.0/tests/test_ruff_config.py

@@ -0,0 +1,61 @@
+import subprocess
+import sys
+import tempfile
+import unittest
+from pathlib import Path
+
+
+class RuffConfigurationTests(unittest.TestCase):
+    def check_source(self, source):
+        with tempfile.TemporaryDirectory() as temp:
+            root = Path(temp)
+            path = root / "sample.py"
+            path.write_text(source, encoding="utf-8", newline="\n")
+            checked = subprocess.run(
+                [sys.executable, "-m", "nk_python", "check", str(path)],
+                cwd=root,
+                capture_output=True,
+                text=True,
+                check=False,
+            )
+            self.assertEqual(path.read_bytes(), source.encode("utf-8"))
+            return checked.returncode, checked.stdout + checked.stderr
+
+    def test_check_ignores_only_the_disabled_rules(self):
+        source = (
+            "import warnings\n\n\n"
+            "def factory():\n    return object()\n\n\n"
+            "def with_default(value=factory()):\n    return value\n\n\n"
+            "def repeated():\n    return 1\n\n\n"
+            "def repeated():\n    return 2\n\n\n"
+            'warnings.warn("example")\n'
+            "pairs = zip([1], [2])\n"
+        )
+        status, output = self.check_source(source)
+        self.assertEqual(status, 0, output)
+        self.assertIn("All checks passed!", output)
+        for rule in ("B008", "B028", "B905", "F811"):
+            with self.subTest(rule=rule):
+                self.assertNotIn(rule, output)
+
+    def test_check_preserves_other_rules(self):
+        source = (
+            "import os\nfrom typing import List\n\n"
+            "missing_name\n"
+            "42\n"
+            "for unused in range(2):\n"
+            '    print("example")\n'
+            "callbacks = []\n"
+            "for value in range(2):\n"
+            "    callbacks.append(lambda: value)\n"
+            "identity = lambda value: value\n"
+        )
+        status, output = self.check_source(source)
+        self.assertEqual(status, 1, output)
+        for rule in ("F821", "B023", "F401", "B018", "B007", "UP035", "E731"):
+            with self.subTest(rule=rule):
+                self.assertRegex(output, rf": {rule}\b")
+
+
+if __name__ == "__main__":
+    unittest.main()

{nk_python_style-0.1.0 → nk_python_style-0.2.0}/tests/test_rules.py

@@ -3,7 +3,7 @@ import unittest
 from pathlib import Path
 
 from nk_python.ast_checks import check_source
-from nk_python.comment_checks import LANGUAGE_AVAILABLE, check_comments
+from nk_python.comment_checks import check_comments
 from nk_python.structure_checks import discover
 
 
@@ -43,9 +43,8 @@ class RuleTests(unittest.TestCase):
 
     def test_inline_comment_and_language(self):
         self.assertIn("PY005", codes("value = 1  # a comment next to code\n"))
-        if LANGUAGE_AVAILABLE:
-            self.assertIn("PY007", codes("# Cette fonction calcule la valeur finale pour chaque entrée.\nvalue = 1\n"))
-            self.assertNotIn("PY007", codes("# This function computes the final value for every input.\nvalue = 1\n"))
+        self.assertIn("PY007", codes("# Cette fonction calcule la valeur finale pour chaque entrée.\nvalue = 1\n"))
+        self.assertNotIn("PY007", codes("# This function computes the final value for every input.\nvalue = 1\n"))
 
     def test_trailing_comment_stays_in_function_scope(self):
         source = "def item():\n    return 1\n    # trailing inside function\n# outside function\n"