Skip to content

ci: fix Codecov upload by untracking integrationtests before removal#5511

Merged
marten-seemann merged 1 commit intomasterfrom
codecov-remove-integrationtests
Dec 29, 2025
Merged

ci: fix Codecov upload by untracking integrationtests before removal#5511
marten-seemann merged 1 commit intomasterfrom
codecov-remove-integrationtests

Conversation

@marten-seemann
Copy link
Copy Markdown
Member

@marten-seemann marten-seemann commented Dec 29, 2025

Fix Codecov upload in CI by untracking and removing integrationtests in unit.yml

Update the unit workflow to run git rm -r --cached integrationtests before rm -rf integrationtests in unit.yml.

📍Where to Start

Start with the 'Remove integrationtests' step in unit.yml.


Macroscope summarized 054bea3.

Comment thread .github/workflows/unit.yml
Comment thread .github/workflows/unit.yml
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (master@dab9a9e). Learn more about missing BASE report.
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #5511   +/-   ##
=========================================
  Coverage          ?   84.07%           
=========================================
  Files             ?      158           
  Lines             ?    16280           
  Branches          ?        0           
=========================================
  Hits              ?    13686           
  Misses            ?     1962           
  Partials          ?      632           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
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 modifies the GitHub Actions unit test workflow to fix Codecov upload by untracking the integrationtests directory from Git before removing it from the filesystem.

Key Changes:

  • Replaces simple directory removal with a two-step process: first untracking from Git using git rm -r --cached, then removing the directory with rm -rf
  • Removes the shell: bash directive from the step

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/unit.yml
Comment thread .github/workflows/unit.yml
@marten-seemann marten-seemann force-pushed the codecov-remove-integrationtests branch from f912dde to 054bea3 Compare December 29, 2025 14:03
@marten-seemann marten-seemann merged commit f3d315f into master Dec 29, 2025
50 checks passed
@marten-seemann marten-seemann deleted the codecov-remove-integrationtests branch December 29, 2025 14:33
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