fix: correct linting errors in green-house-migration project

matiasz8 · matiasz8 · commit f07b7f472746 · 2025-09-01T02:54:42.000-03:00
- Fixed hadolint errors in Dockerfile:
  * Specified exact Python version (3.12.7-alpine)
  * Separated ENV statements instead of using backslashes
  * Added --no-cache-dir to apk add and pip install commands

- Fixed stylelint errors in style.css:
  * Changed #ffffff to #fff for shorter hex color
  * Changed rgba(0, 0, 0, 0.1) to rgba(0, 0, 0, 10%) for modern color notation
  * Reordered selectors to fix specificity issues (.sidebar-menu a before .site-title a:hover)

- Fixed htmlhint errors in index.html:
  * Changed DOCTYPE to lowercase
  * Moved title element to be first in head section
diff --git a/examples/green-house-migration/Dockerfile b/examples/green-house-migration/Dockerfile
@@ -1,14 +1,14 @@
 # Single stage Dockerfile for Greenhouse API Proxy
-FROM python:3.12-alpine
+FROM python:3.12.7-alpine
 
 # Set environment variables
-ENV PYTHONDONTWRITEBYTECODE=1 \
-    PYTHONUNBUFFERED=1 \
-    PYTHONPATH=/app \
-    PORT=8000
+ENV PYTHONDONTWRITEBYTECODE=1
+ENV PYTHONUNBUFFERED=1
+ENV PYTHONPATH=/app
+ENV PORT=8000
 
 # Install system dependencies
-RUN apk add --no-cache \
+RUN apk add --no-cache-dir \
     gcc \
     musl-dev \
     libffi-dev \
@@ -17,7 +17,7 @@ RUN apk add --no-cache \
     git
 
 # Install pipenv
-RUN pip install pipenv
+RUN pip install --no-cache-dir pipenv
 
 # Create non-root user
 RUN addgroup -g 1000 appuser && adduser -D -s /bin/sh -u 1000 -G appuser appuser
diff --git a/examples/green-house-migration/config.py b/examples/green-house-migration/config.py
@@ -7,22 +7,22 @@
 
 
 def get_required_env(key: str) -> str:
-    """Obtener variable de entorno requerida."""
-    value = os.getenv(key)
-    if not value:
-        raise ValueError(f"Variable de entorno {key} es requerida")
-    return value
+  """Obtener variable de entorno requerida."""
+  value = os.getenv(key)
+  if not value:
+    raise ValueError(f"Variable de entorno {key} es requerida")
+  return value
 
 
 def get_optional_env(key: str, default: str = "") -> str:
-    """Obtener variable de entorno opcional."""
-    return os.getenv(key, default)
+  """Obtener variable de entorno opcional."""
+  return os.getenv(key, default)
 
 
 # Configuración de la API
 GREENHOUSE_API_KEY = get_required_env("GREENHOUSE_API_KEY")
 GREENHOUSE_API_URL = get_optional_env(
-    "GREENHOUSE_API_URL", "https://harvest.greenhouse.io/v1"
+  "GREENHOUSE_API_URL", "https://harvest.greenhouse.io/v1"
 )
 
 # Configuración de la aplicación
diff --git a/examples/green-house-migration/dashboard/index.html b/examples/green-house-migration/dashboard/index.html
@@ -1,9 +1,9 @@
-<!DOCTYPE html>
+<!doctype html>
 <html lang="en">
   <head>
+    <title>TeamTailor Sourced Candidates Analytics Dashboard</title>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>TeamTailor Sourced Candidates Analytics Dashboard</title>
     <link
       rel="icon"
       type="image/svg+xml"
diff --git a/examples/green-house-migration/docs/assets/css/style.css b/examples/green-house-migration/docs/assets/css/style.css
@@ -12,7 +12,7 @@ body {
     Ubuntu, Cantarell, sans-serif;
   line-height: 1.6;
   color: #24292e;
-  background-color: #ffffff;
+  background-color: #fff;
 }
 
 /* Header */
@@ -39,6 +39,14 @@ body {
   text-decoration: none;
 }
 
+.sidebar-menu a {
+  color: #0366d6;
+  text-decoration: none;
+  padding: 0.25rem 0;
+  display: block;
+  transition: color 0.2s;
+}
+
 .site-title a:hover {
   color: #f0f0f0;
 }
@@ -68,7 +76,7 @@ body {
   background: white;
   padding: 2rem;
   border-radius: 8px;
-  box-shadow: 0 2px 8px rgba(0, 0, 0, 0.1);
+  box-shadow: 0 2px 8px rgba(0, 0, 0, 10%);
 }
 
 /* Sidebar */
@@ -100,14 +108,6 @@ body {
   margin-bottom: 0.5rem;
 }
 
-.sidebar-menu a {
-  color: #0366d6;
-  text-decoration: none;
-  padding: 0.25rem 0;
-  display: block;
-  transition: color 0.2s;
-}
-
 .sidebar-menu a:hover {
   color: #0056b3;
   text-decoration: underline;
