fix: skip kover-report step on schedule events

[driessamyn]

The mi-kas/kover-report action only supports pull_request and push events, causing the nightly compatibility test workflow to fail.

Summary by CodeRabbit

• Chores
  • Optimized CI/CD workflow to run coverage reports only on relevant events, reducing unnecessary pipeline executions.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 847aefcb-c796-46aa-a36e-742621dd0c1f

📥 Commits

Reviewing files that changed from the base of the PR and between 4cc9255 and a67c5d9.

📒 Files selected for processing (1)

• .github/workflows/build-and-test.yml

---

📝 Walkthrough

Walkthrough

This change adds a conditional filter to the coverage report action in the GitHub Actions workflow, restricting it to run only on pull_request and push events, preventing execution during scheduled or manually-dispatched workflow runs.

Changes

Cohort / File(s)	Summary
Workflow Configuration .github/workflows/build-and-test.yml	Added if: condition to the coverage report step to limit execution to pull_request and push event types.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A conditional hop, so light and so neat,
Coverage reports now find their proper beat,
No more on schedules shall actions run free,
Just push and pull requests—efficient, you see!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: adding a conditional to skip the kover-report step during schedule events, which directly addresses the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/kover-report-schedule

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

Unit Tests

 61 files  ±0   61 suites  ±0   3m 1s ⏱️ -14s
545 tests ±0  545 ✅ ±0  0 💤 ±0  0 ❌ ±0 
561 runs  ±0  561 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit a67c5d9. ± Comparison against base commit 4cc9255.

This pull request removes 39 and adds 39 tests. Note that renamed tests count towards both.

net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [10] LONGVARCHAR, "LONGVARCHAR", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1105/0x00007f39e4587100@6755ec2e
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [11] NCHAR, "NCHAR", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1106/0x00007f39e4587320@3dba6adc
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [12] INSTANT, net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$949/0x00007f39e454bcc0@51cba6b9
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [12] NCLOB, "NCLOB", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1107/0x00007f39e4587540@2bced423
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [13] DATE, net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$950/0x00007f39e4554000@5fdfc6c
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [13] NVARCHAR, "NVARCHAR", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1108/0x00007f39e4587760@3ecc0fd0
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [14] LOCALDATE, net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$951/0x00007f39e4554220@5a5f044d
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [14] ROWID, "ROWID", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1109/0x00007f39e4587980@1480e0d
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [15] LOCALDATETIME, net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$952/0x00007f39e4554440@5fbb4e80
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [15] SQLXML, "SQLXML", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1110/0x00007f39e4587ba0@d1b7ac3
…

net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [10] LONGVARCHAR, "LONGVARCHAR", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1141/0x00007f8da0582190@44ab98c6
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [11] NCHAR, "NCHAR", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1142/0x00007f8da05823b0@624d07bc
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [12] INSTANT, net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$983/0x00007f8da0545d48@49cc8144
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [12] NCLOB, "NCLOB", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1143/0x00007f8da05825d0@1c8b1a0f
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [13] DATE, net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$984/0x00007f8da0548000@70b5eec2
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [13] NVARCHAR, "NVARCHAR", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1144/0x00007f8da05827f0@59316d3
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [14] LOCALDATE, net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$985/0x00007f8da0548220@53a9a97d
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [14] ROWID, "ROWID", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1145/0x00007f8da0582a10@71dd1cc2
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [15] LOCALDATETIME, net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$986/0x00007f8da0548440@e9ae359
net.samyn.kapper.internal.automapper.SQLTypesConverterTest ‑ [15] SQLXML, "SQLXML", net.samyn.kapper.internal.automapper.SQLTypesConverterTest$Companion$$Lambda$1146/0x00007f8da0582c30@8d88351
…

[github-actions]

Code Coverage

Total Project Coverage	98.37%

[github-actions]

POSTGRESQL Integration Tests

11 files  ±0  11 suites  ±0   24s ⏱️ -1s
51 tests ±0  51 ✅ ±0  0 💤 ±0  0 ❌ ±0 
81 runs  ±0  81 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit a67c5d9. ± Comparison against base commit 4cc9255.

[github-actions]

DUCKDB Integration Tests

11 files  ±0  11 suites  ±0   24s ⏱️ -1s
51 tests ±0  51 ✅ ±0  0 💤 ±0  0 ❌ ±0 
81 runs  ±0  81 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit a67c5d9. ± Comparison against base commit 4cc9255.

[github-actions]

SQLITE Integration Tests

11 files  ±0  11 suites  ±0   26s ⏱️ +2s
51 tests ±0  46 ✅ ±0  5 💤 ±0  0 ❌ ±0 
81 runs  ±0  76 ✅ ±0  5 💤 ±0  0 ❌ ±0 

Results for commit a67c5d9. ± Comparison against base commit 4cc9255.

[github-actions]

MSSQLSERVER Integration Tests

11 files  ±0  11 suites  ±0   39s ⏱️ -7s
51 tests ±0  46 ✅ ±0  5 💤 ±0  0 ❌ ±0 
81 runs  ±0  76 ✅ ±0  5 💤 ±0  0 ❌ ±0 

Results for commit a67c5d9. ± Comparison against base commit 4cc9255.

[github-actions]

MYSQL Integration Tests

11 files  ±0  11 suites  ±0   49s ⏱️ +5s
51 tests ±0  46 ✅ ±0  5 💤 ±0  0 ❌ ±0 
81 runs  ±0  76 ✅ ±0  5 💤 ±0  0 ❌ ±0 

Results for commit a67c5d9. ± Comparison against base commit 4cc9255.

[github-actions]

ORACLE Integration Tests

11 files  ±0  11 suites  ±0   1m 11s ⏱️ +3s
51 tests ±0  46 ✅ ±0  5 💤 ±0  0 ❌ ±0 
81 runs  ±0  76 ✅ ±0  5 💤 ±0  0 ❌ ±0 

Results for commit a67c5d9. ± Comparison against base commit 4cc9255.