Add a few lint rules, set line-length to 120 (#23)

* Remove unused imports

* Add __all__

* Max line length

* New rules

* Fix linting errors

* Update pre-commit hook

* Move __all__ definition

Author: AdrienVannson

.pre-commit-config.yaml

@@ -8,18 +8,11 @@ repos:
       - id: ruff-format
         args: ["--diff", "src", "tests"]
       - id: ruff
-        args: ["--select", "I", "src", "tests"]
+        args: ["src", "tests"]
 
   - repo: https://github.com/PyCQA/doc8
     rev: 0.10.1
     hooks:
       - id: doc8
         additional_dependencies:
           - toml
-
-  - repo: https://github.com/macisamuele/language-formatters-pre-commit-hooks
-    rev: v2.10.0
-    hooks:
-      - id: pretty-format-java
-        args: [--autofix, --aosp]
-        files: ^.*\.java$

docs/conf.py

@@ -12,7 +12,6 @@
 
 import toml
 
-
 # -- Project information -----------------------------------------------------
 
 project = "betterproto"

pyproject.toml

@@ -53,10 +53,25 @@ rust-codec = ["betterproto-rust-codec"]
 [tool.ruff]
 extend-exclude = ["tests/output_*"]
 target-version = "py38"
+line-length = 120
+
+[tool.ruff.lint]
+select = [
+    "F401",  # Unused imports
+    "F841",  # Unused local variables
+    "F821",  # Undefined names
+    "E501",  # Line length violations
+
+    "SIM101", # Simplify unnecessary if-else blocks
+    "SIM102", # Simplify return or yield statements
+    "SIM103", # Simplify list/set/dict comprehensions
+
+    "I",
+]
+
 
 [tool.ruff.lint.isort]
 combine-as-imports = true
-lines-after-imports = 2
 
 # Dev workflow tasks