Skip to content

Conversation

@giovgiac
Copy link
Contributor

Summary

  • replace the deprecated coverage-badge dependency with genbadge and enable the required extras
  • update the backend CI workflow to collect flake8 output and generate local tests, flake8, and coverage badges via genbadge
  • surface the new badges in the backend README and document how to recreate them in the backend testing guide

Testing

  • poetry run pytest --junitxml=pytest-results.xml --cov=human_evaluation_tool --cov-report=term-missing --cov-report=xml

https://chatgpt.com/codex/tasks/task_b_68e8d4b5bc648323975493f05c54f40f

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting

Comment on lines +119 to +124
- name: Generate status badges
working-directory: backend
run: |
poetry run genbadge tests --local -i pytest-results.xml -o tests.svg
poetry run genbadge flake8 --local -i flake8.log -o flake8.svg
poetry run genbadge coverage --local -i coverage.xml -o coverage.svg

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Ensure genbadge dependency is installed

The workflow now runs poetry run genbadge … to generate SVG badges, but the commit only adds genbadge to backend/pyproject.toml and does not update backend/poetry.lock. The CI job installs dependencies from the lock file (poetry install --with dev), so genbadge is never installed and these steps will fail with Command not found. Run poetry lock (or otherwise update the lock file) so the package is present in CI before invoking it.

Useful? React with 👍 / 👎.

Base automatically changed from codex/add-github-actions-for-ci-pipeline to main October 15, 2025 03:15
@giovgiac giovgiac closed this Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants