Skip to content

Skip tests to unblock PR CI#967

Merged
scott-routledge2 merged 4 commits intomainfrom
scott/ci_test_fixes
Dec 10, 2025
Merged

Skip tests to unblock PR CI#967
scott-routledge2 merged 4 commits intomainfrom
scott/ci_test_fixes

Conversation

@scott-routledge2
Copy link
Contributor

@scott-routledge2 scott-routledge2 commented Dec 10, 2025

Changes included in this PR

  • Moves Iceberg DDL / dict encoding tests to Nightly to free up disk space
  • Skips flakey Ollama test with TODO

Testing strategy

User facing changes

Checklist

  • Pipelines passed before requesting review. To run CI you must include [run CI] in your commit message.
  • I am familiar with the Contributing Guide
  • I have installed + ran pre-commit hooks.

@codecov
Copy link

codecov bot commented Dec 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.79%. Comparing base (c33fbb5) to head (ebbbc72).
⚠️ Report is 157 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #967      +/-   ##
==========================================
+ Coverage   66.68%   68.79%   +2.10%     
==========================================
  Files         186      195       +9     
  Lines       66795    67681     +886     
  Branches     9507     9627     +120     
==========================================
+ Hits        44543    46559    +2016     
+ Misses      19572    18292    -1280     
- Partials     2680     2830     +150     

@scott-routledge2 scott-routledge2 marked this pull request as ready for review December 10, 2025 21:30
@scott-routledge2 scott-routledge2 merged commit f89befd into main Dec 10, 2025
49 of 56 checks passed
@scott-routledge2 scott-routledge2 deleted the scott/ci_test_fixes branch December 10, 2025 23:01
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