Skip to content

fix(quality): Apply GitHub Copilot suggestions for code quality improvements - CGAI-25#9

Merged
Yosoyepa merged 2 commits intodevelopfrom
feature/update-docs
Nov 7, 2025
Merged

fix(quality): Apply GitHub Copilot suggestions for code quality improvements - CGAI-25#9
Yosoyepa merged 2 commits intodevelopfrom
feature/update-docs

Conversation

@Yosoyepa
Copy link
Owner

@Yosoyepa Yosoyepa commented Nov 7, 2025

Container & Infrastructure Fixes

  • Fix Dockerfile healthcheck: use curl instead of Python requests dependency
  • Add curl to system dependencies in Docker image
  • Fix Redis healthcheck command in docker-compose.yml
  • Update Redis URL to include password authentication

CI/CD Workflow Improvements

  • Change lint workflow to check formatting instead of modifying code
  • Fix Black: use --check flag for validation only
  • Fix isort: use --check-only flag for validation only
  • Add docker-compose installation to GitHub Actions workflow

Dependency Management

  • Create requirements-dev.txt for development dependencies
  • Move black, isort, mypy, pytest, pytest-asyncio, pytest-cov to dev requirements
  • Update requirements.txt to contain only production dependencies

Code Quality

  • Remove unused pytest import from test_main.py

Related: CGAI-25

… CGAI-25

- Install docker-compose before running docker-compose config validation
- Resolve "command not found" error in GitHub Actions runner
- Ensures CI/CD pipeline can validate docker-compose.yml syntax

Related: CGAI-25
…vements - CGAI-25

- Fix Dockerfile healthcheck to use curl instead of Python requests
- Fix Redis healthcheck and URL authentication in docker-compose.yml
- Change lint workflow to check formatting instead of modifying code
- Separate development dependencies into requirements-dev.txt
- Remove unused pytest import from test_main.py

Related: CGAI-25
@Yosoyepa Yosoyepa merged commit e3a177c into develop Nov 7, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant