Skip to content

Conversation

Pouyanpi
Copy link
Collaborator

Remove branch restriction from pull_request trigger to allow codecov coverage reporting on PRs targeting any branch, not just develop.

@Pouyanpi Pouyanpi requested a review from Copilot September 16, 2025 10:22
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enables codecov coverage reporting for all pull requests by removing the branch restriction that previously limited coverage reports to PRs targeting only the develop branch.

  • Removes branch filter from pull_request trigger in the test coverage workflow

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Pouyanpi Pouyanpi changed the base branch from develop to chore/type-clean-llm September 16, 2025 10:23
@Pouyanpi Pouyanpi changed the base branch from chore/type-clean-llm to develop September 16, 2025 10:24
@codecov-commenter
Copy link

codecov-commenter commented Sep 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.66%. Comparing base (7f70ee7) to head (38fe438).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1402   +/-   ##
========================================
  Coverage    71.66%   71.66%           
========================================
  Files          171      171           
  Lines        17015    17015           
========================================
  Hits         12193    12193           
  Misses        4822     4822           
Flag Coverage Δ
python 71.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Remove branch restriction from pull_request trigger to allow codecov
coverage reporting on PRs targeting any branch, not just develop.
@Pouyanpi Pouyanpi force-pushed the ci/codecov-restriction branch from d634144 to 38fe438 Compare September 16, 2025 15:47
@Pouyanpi Pouyanpi added this to the v0.17.0 milestone Sep 16, 2025
@Pouyanpi Pouyanpi added the CI label Sep 16, 2025
Copy link
Collaborator

@tgasser-nv tgasser-nv left a comment

Choose a reason for hiding this comment

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

Looks great, please merge. We should have this coverage on all the PRs not just those headed to develop. Thanks!

@Pouyanpi Pouyanpi merged commit 5d974e5 into develop Sep 18, 2025
12 checks passed
@Pouyanpi Pouyanpi deleted the ci/codecov-restriction branch September 18, 2025 13:12
tgasser-nv pushed a commit that referenced this pull request Sep 22, 2025
Remove branch restriction from pull_request trigger to allow codecov
coverage reporting on PRs targeting any branch, not just develop.
tgasser-nv pushed a commit that referenced this pull request Sep 22, 2025
Remove branch restriction from pull_request trigger to allow codecov
coverage reporting on PRs targeting any branch, not just develop.
Pouyanpi added a commit that referenced this pull request Oct 1, 2025
Remove branch restriction from pull_request trigger to allow codecov
coverage reporting on PRs targeting any branch, not just develop.
tgasser-nv pushed a commit that referenced this pull request Oct 14, 2025
Remove branch restriction from pull_request trigger to allow codecov
coverage reporting on PRs targeting any branch, not just develop.
tgasser-nv pushed a commit that referenced this pull request Oct 14, 2025
Remove branch restriction from pull_request trigger to allow codecov
coverage reporting on PRs targeting any branch, not just develop.
tgasser-nv pushed a commit that referenced this pull request Oct 14, 2025
Remove branch restriction from pull_request trigger to allow codecov
coverage reporting on PRs targeting any branch, not just develop.
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.

3 participants