Skip to content

Conversation

@tassadit-sadoun
Copy link
Owner

Cette PR contient :

  • La version fonctionnelle de l’API conformément au périmètre initial.

  • Un fichier IMPROVEMENTS.md recensant les améliorations potentielles identifiées (tests, sécurité, validations, performances, etc.).

  • Une mise à jour du README.md incluant un lien vers ces pistes.

  • Faute de temps, certaines améliorations n’ont pas été implémentées mais sont documentées pour faciliter leur ajout ultérieur.

tassadit added 30 commits May 20, 2025 17:16
…and env vars, mount shared_data volume

- Define container name for easier identification
- Modify exposed port for better compatibility
- Add .env file support and environment variables injection
- Mount shared_data directory as volume for persistent shared storage
- switch to slim base image for reduced image size
- copy requirements.txt first to leverage Docker layer caching
- add pip upgrade and --no-cache-dir to minimize image footprint
- define default CMD to run Celery worker with info log level
- Add core dependencies: fastapi, celery, pandas, psycopg[binary], redis, requests, SQLAlchemy, uvicorn, pydantic[email]
- Include testing tools: pytest, pytest-cov, coverage, httpx
- Add additional libraries: importlib-metadata, python-multipart, starlette
- Token creation and verification logic
- Loaded simulated credentials and secret key from environment variables
tassadit added 5 commits May 25, 2025 15:29
- Included detailed pytest summary showing all 15 tests passed
- Tests cover API routes and asynchronous import task handling
- Confirms robustness of client import, pagination, and error cases
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