Skip to content

Conversation

@ishymko
Copy link
Member

@ishymko ishymko commented Dec 16, 2025

Description

The job added in #262 is failing on main as it tries to post a PR comment although there is no PR 🤦 .

Fixed by:

  1. Add conditions for PR related steps.
  2. Bring back publishing coverage report as artifact removed mistakenly in ci: post code coverage delta comment on PRs #262.
  3. Add a link to this artifact to code coverage comment on PRs.

@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@ishymko ishymko force-pushed the ishymko/fix-coverage-workflow branch from 84de9f1 to 91b8a53 Compare December 16, 2025 11:15
@github-actions
Copy link

github-actions bot commented Dec 16, 2025

🧪 Code Coverage

⬇️ Download HTML Report

Metric Main PR Delta
lines 81.57% 81.57% ⚪️ 0.00%
statements 81.33% 81.33% ⚪️ 0.00%
functions 87.18% 87.18% ⚪️ 0.00%
branches 69.98% 69.98% ⚪️ 0.00%

Generated by unit-tests.yml

@ishymko ishymko force-pushed the ishymko/fix-coverage-workflow branch from 91b8a53 to ab61a5c Compare December 16, 2025 11:17
@ishymko ishymko force-pushed the ishymko/fix-coverage-workflow branch from ab61a5c to 9b692ad Compare December 16, 2025 11:19
@ishymko ishymko marked this pull request as ready for review December 16, 2025 11:19
@ishymko ishymko requested a review from a team as a code owner December 16, 2025 11:19
@ishymko ishymko enabled auto-merge (squash) December 16, 2025 11:23
@ishymko ishymko merged commit cf9c189 into main Dec 16, 2025
14 checks passed
@ishymko ishymko deleted the ishymko/fix-coverage-workflow branch December 16, 2025 12:18
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.

3 participants