Skip to content

Conversation

@joseph-isaacs
Copy link
Contributor

No description provided.

Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
@joseph-isaacs joseph-isaacs changed the title wip chore: use assert_arrays_eq! instead of using buffer eq Jan 14, 2026
@joseph-isaacs joseph-isaacs added the chore Release label indicating a trivial change label Jan 14, 2026
@joseph-isaacs joseph-isaacs marked this pull request as ready for review January 14, 2026 15:14
@codecov
Copy link

codecov bot commented Jan 14, 2026

Codecov Report

❌ Patch coverage is 96.44970% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.39%. Comparing base (f1684e6) to head (3ef43ee).
⚠️ Report is 5 commits behind head on develop.

Files with missing lines Patch % Lines
vortex-array/src/arrays/listview/tests/filter.rs 0.00% 6 Missing ⚠️
vortex-array/src/arrays/listview/tests/take.rs 0.00% 6 Missing ⚠️

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@joseph-isaacs joseph-isaacs changed the title chore: use assert_arrays_eq! instead of using buffer eq chore: use assert_arrays_eq! instead of using buffer eq Jan 14, 2026
Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
@joseph-isaacs joseph-isaacs enabled auto-merge (squash) January 14, 2026 17:27
@joseph-isaacs joseph-isaacs merged commit 6127f54 into develop Jan 14, 2026
47 of 48 checks passed
@joseph-isaacs joseph-isaacs deleted the ji/update-test-assert-eq branch January 14, 2026 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Release label indicating a trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants