Skip to content

Conversation

rzabarazesh
Copy link
Collaborator

@rzabarazesh rzabarazesh commented Sep 12, 2025

Purpose

This is needed for vllm-project/ci-infra#164 and used to enable coverage report generation using pytest-cov.

Test Plan

Test Result


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

@mergify mergify bot added the ci/build label Sep 12, 2025
Signed-off-by: Reza Barazesh <rezabarazesh@meta.com>
Copy link

pytorch-bot bot commented Sep 14, 2025

No ciflow labels are configured for this repo.
For information on how to enable CIFlow bot see this wiki

@rzabarazesh rzabarazesh changed the title [do not merge] Test pytest-cov with jinja Add pytest-cov and .coveragerc Sep 14, 2025
@rzabarazesh rzabarazesh added the ready ONLY add when PR is ready to merge/full CI is needed label Sep 14, 2025
@pytorch-bot pytorch-bot bot removed the ci/build label Sep 14, 2025
@mergify mergify bot added the ci/build label Sep 14, 2025
Copy link

pytorch-bot bot commented Sep 14, 2025

No ciflow labels are configured for this repo.
For information on how to enable CIFlow bot see this wiki

@rzabarazesh rzabarazesh marked this pull request as ready for review September 15, 2025 13:01
@mgoin mgoin merged commit d96e111 into vllm-project:main Sep 16, 2025
80 of 82 checks passed
FeiDaLI pushed a commit to FeiDaLI/vllm that referenced this pull request Sep 25, 2025
Signed-off-by: Reza Barazesh <rezabarazesh@meta.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/build ready ONLY add when PR is ready to merge/full CI is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants