Skip to content

fix: add 'reach' option comment layout for codecov configuration#173

Merged
reddevilmidzy merged 1 commit intomainfrom
codecov-patch
Jul 15, 2025
Merged

fix: add 'reach' option comment layout for codecov configuration#173
reddevilmidzy merged 1 commit intomainfrom
codecov-patch

Conversation

@reddevilmidzy
Copy link
Owner

♟️ What’s this PR about?

PR에 codecov가 코멘트를 달아주지만 전체 커버리지를 보여주지 않아 해당 설정을 추가하였습니다.

🔗 Related Issues / PRs

part of #167

Copilot AI review requested due to automatic review settings July 15, 2025 07:12
@github-actions github-actions bot added this to the 5차 MVP 구현 milestone Jul 15, 2025
Copy link
Contributor

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 updates the Codecov comment configuration to include branch reach information and enable carryforward flags in coverage comments.

  • Adds the reach token to the comment layout so reach coverage is displayed.
  • Enables show_carryforward_flags to surface any carryforward flags in the comment.
Comments suppressed due to low confidence (2)

.github/codecov.yml:26

  • [nitpick] Consider converting the comma-separated layout string into a YAML sequence (e.g., `layout:
    • header
    • reach
    • diff
      ...`) to improve readability and ease future updates.
  layout: "header, reach, diff, flags, files, footer"

.github/codecov.yml:32

  • [nitpick] Add a comment explaining what show_carryforward_flags does and when it should be used, so future maintainers understand its effect on the Codecov comment output.
  show_carryforward_flags: true

@reddevilmidzy reddevilmidzy merged commit f7f3cfc into main Jul 15, 2025
7 checks passed
@reddevilmidzy reddevilmidzy deleted the codecov-patch branch July 15, 2025 07:28
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