Skip to content

fix: requirements-api.txt tested in ci#1258

Merged
inimaz merged 2 commits into
masterfrom
fix/fastapi-starlette-2
Jun 28, 2026
Merged

fix: requirements-api.txt tested in ci#1258
inimaz merged 2 commits into
masterfrom
fix/fastapi-starlette-2

Conversation

@inimaz

@inimaz inimaz commented Jun 28, 2026

Copy link
Copy Markdown
Collaborator

Description

Fix issues like this one https://github.com/mlco2/codecarbon/actions/runs/28317928989/job/83894562865. Where in local and ci works because they use the pyproject.toml, but the server uses requirements/requirements-api.txt so it fails.

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

AI Usage Disclosure

Please refer to docs/how-to/ai-policy.md for detailed guidelines on how to disclose AI usage in your PR. Accurately completing this section is mandatory.

  • 🟥 AI-vibecoded: You cannot explain the logic. Car analogy : the car drive by itself, you are outside it and just tell it where to go.
  • 🟠 AI-generated: Car analogy : the car drive by itself, you are inside and give instructions.
  • ⭐ AI-assisted. Car analogy : you drive the car, AI help you find your way.
  • ♻️ No AI used. Car analogy : you drive the car.

@inimaz inimaz requested a review from a team as a code owner June 28, 2026 10:38
@inimaz inimaz merged commit aea5d20 into master Jun 28, 2026
8 checks passed
@inimaz inimaz deleted the fix/fastapi-starlette-2 branch June 28, 2026 10:50
@benoit-cty benoit-cty mentioned this pull request Jun 30, 2026
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.

2 participants