Add linting and type checking to CI workflow

Author: Mehdy Chaillou

.github/workflows/build.yml

@@ -11,8 +11,36 @@ on:
   workflow_dispatch:
 
 jobs:
+  lint:
+    name: Lint & Type Check
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.11'
+
+      - name: Install lint dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install ruff pyright
+          pip install -r python/requirements.txt
+
+      - name: Run ruff linter
+        working-directory: python
+        run: ruff check .
+
+      - name: Run pyright type checker
+        working-directory: python
+        run: pyright
+
   build-python-windows:
     name: Build Python (Windows)
+    needs: [lint]
     runs-on: windows-latest
 
     steps:
@@ -46,6 +74,7 @@ jobs:
 
   build-python-linux:
     name: Build Python (Linux)
+    needs: [lint]
     runs-on: ubuntu-latest
 
     steps:
@@ -84,6 +113,7 @@ jobs:
 
   build-csharp:
     name: Build C# (Windows)
+    needs: [lint]
     runs-on: windows-latest
 
     steps:
@@ -102,7 +132,7 @@ jobs:
 
       - name: Build solution
         working-directory: csharp
-        run: msbuild StarCitizenPlaytimeCalculator.sln /p:Configuration=Release /p:Platform="Any CPU"
+        run: msbuild StarCitizenPlaytimeCalculator.sln /p:Configuration=Release /p:Platform="Any CPU" /p:TreatWarningsAsErrors=true /p:RunAnalyzersDuringBuild=true
 
       - name: Upload C# artifact
         uses: actions/upload-artifact@v4

python/pyproject.toml

@@ -0,0 +1,24 @@
+[tool.ruff]
+line-length = 120
+target-version = "py311"
+
+[tool.ruff.lint]
+select = [
+    "E",      # pycodestyle errors
+    "F",      # pyflakes
+    "W",      # pycodestyle warnings
+    "B",      # flake8-bugbear
+    "I",      # isort
+]
+ignore = [
+    "E501",   # line too long (handled by formatter)
+]
+
+[tool.pyright]
+pythonVersion = "3.11"
+typeCheckingMode = "basic"
+reportMissingImports = true
+reportMissingTypeStubs = false
+reportUnusedImport = true
+reportUnusedVariable = true
+reportAttributeAccessIssue = true