chore: enable Flake8 on VS Code and align with Black rules

Author: nanotaboada

.vscode/extensions.json

@@ -1,7 +1,7 @@
 {
   "recommendations": [
     "ms-python.python",
-    "ms-python.vscode-pylance",
+    "ms-python.flake8",
     "ms-python.black-formatter",
     "github.vscode-pull-request-github",
     "github.vscode-github-actions",

.vscode/settings.json

@@ -1,7 +1,4 @@
 {
-  "python.testing.unittestEnabled": false,
-  "python.testing.pytestEnabled": true,
-  "python.testing.pytestArgs": ["tests"],
   "files.exclude": {
     "**/__pycache__": true,
     "**/.git": true,
@@ -13,6 +10,44 @@
     "editor.defaultFormatter": "ms-python.black-formatter",
     "editor.formatOnSave": true
   },
+  "python.testing.unittestEnabled": false,
+  "python.testing.pytestEnabled": true,
+  "python.testing.pytestArgs": ["tests"],
+  "flake8.enabled": true,
+  "flake8.importStrategy": "fromEnvironment",
+  "flake8.path": ["${interpreter}", "-m", "flake8"],
+  // Point flake8 to use your existing config file automatically
+  "flake8.args": [
+    "--max-line-length=88",
+    "--max-complexity=10",
+    "--select=E,F,W",
+    "--extend-ignore=E203,W503",
+    "--exclude=.venv",
+    "--per-file-ignores=tests/test_main.py:E501"
+  ],
+  // Exclude files/folders you don’t want to lint (matching Black’s exclude)
+  "flake8.ignorePatterns": [
+    "**/.git/**",
+    "**/.github/**",
+    "**/.pytest_cache/**",
+    "**/.venv/**",
+    "**/.vscode/**",
+    "**/assets/**",
+    "**/htmlcov/**",
+    "**/postman_collections/**",
+    "**/scripts/**",
+    "**/storage/**",
+    "**/__pycache__/**",
+    "**/tests/test_main.py"
+  ],
+  "flake8.severity": {
+    "convention": "Information",
+    "error": "Error",
+    "fatal": "Error",
+    "refactor": "Hint",
+    "warning": "Warning",
+    "info": "Information"
+  },
   "sonarlint.connectedMode.project": {
     "connectionId": "nanotaboada",
     "projectKey": "nanotaboada_python-samples-fastapi-restful"

models/player_model.py

@@ -23,7 +23,8 @@ class MainModel(BaseModel):
         model_config (ConfigDict): Configuration for Pydantic models, including:
             alias_generator (function): A function to generate field aliases.
                 Here, it uses `to_camel` to convert field names to camelCase.
-            populate_by_name (bool): Allows population of fields by name when using Pydantic models.
+            populate_by_name (bool): Allows population of fields by name when using
+            Pydantic models.
     """
 
     model_config = ConfigDict(alias_generator=to_camel, populate_by_name=True)
@@ -41,10 +42,12 @@ class PlayerModel(MainModel):
         date_of_birth (Optional[str]): The date of birth of the Player, if provided.
         squad_number (int): The unique squad number assigned to the Player.
         position (str): The playing position of the Player.
-        abbr_position (Optional[str]): The abbreviated form of the Player's position, if any.
+        abbr_position (Optional[str]): The abbreviated form of the Player's position,
+        if any.
         team (Optional[str]): The team to which the Player belongs, if any.
         league (Optional[str]): The league where the team plays, if any.
-        starting11 (Optional[bool]): Indicates if the Player is in the starting 11, if provided.
+        starting11 (Optional[bool]): Indicates if the Player is in the starting 11,
+        if provided.
     """
 
     id: int

routes/health_route.py

@@ -13,6 +13,7 @@
 @api_router.get("/health", tags=["Health"])
 async def health_check():
     """
-    Simple health check endpoint. Returns a JSON response with a single key "status" and value "ok".
+    Simple health check endpoint.
+    Returns a JSON response with a single key "status" and value "ok".
     """
     return {"status": "ok"}

routes/player_route.py

@@ -49,7 +49,8 @@ async def post_async(
     Endpoint to create a new player.
 
     Args:
-        player_model (PlayerModel): The Pydantic model representing the Player to create.
+        player_model (PlayerModel): The Pydantic model representing the Player to
+        create.
         async_session (AsyncSession): The async version of a SQLAlchemy ORM session.
 
     Raises:
@@ -115,7 +116,8 @@ async def get_by_id_async(
         PlayerModel: The Pydantic model representing the matching Player.
 
     Raises:
-        HTTPException: Not found error if the Player with the specified ID does not exist.
+        HTTPException: Not found error if the Player with the specified ID does not
+        exist.
     """
     player = await player_service.retrieve_by_id_async(async_session, player_id)
     if not player:
@@ -145,7 +147,8 @@ async def get_by_squad_number_async(
         PlayerModel: The Pydantic model representing the matching Player.
 
     Raises:
-        HTTPException: HTTP 404 Not Found error if the Player with the specified Squad Number does not exist.
+        HTTPException: HTTP 404 Not Found error if the Player with the specified
+        Squad Number does not exist.
     """
     player = await player_service.retrieve_by_squad_number_async(
         async_session, squad_number
@@ -174,11 +177,13 @@ async def put_async(
 
     Args:
         player_id (int): The ID of the Player to update.
-        player_model (PlayerModel): The Pydantic model representing the Player to update.
+        player_model (PlayerModel): The Pydantic model representing the Player to
+        update.
         async_session (AsyncSession): The async version of a SQLAlchemy ORM session.
 
     Raises:
-        HTTPException: HTTP 404 Not Found error if the Player with the specified ID does not exist.
+        HTTPException: HTTP 404 Not Found error if the Player with the specified ID
+        does not exist.
     """
     player = await player_service.retrieve_by_id_async(async_session, player_id)
     if not player:
@@ -208,7 +213,8 @@ async def delete_async(
         async_session (AsyncSession): The async version of a SQLAlchemy ORM session.
 
     Raises:
-        HTTPException: HTTP 404 Not Found error if the Player with the specified ID does not exist.
+        HTTPException: HTTP 404 Not Found error if the Player with the specified ID
+        does not exist.
     """
     player = await player_service.retrieve_by_id_async(async_session, player_id)
     if not player: