Skip to content

Conversation

@connortsui20
Copy link
Contributor

This makes it easier to read the code since the immutable vs mutable names are so similar.

This is a purely cosmetic change

Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
@connortsui20 connortsui20 added the changelog/chore A trivial change label Nov 5, 2025
@connortsui20 connortsui20 requested review from a10y and gatesn November 5, 2025 16:23
@codecov
Copy link

codecov bot commented Nov 5, 2025

Codecov Report

❌ Patch coverage is 26.89655% with 106 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.87%. Comparing base (28f5b3d) to head (b8364c6).
⚠️ Report is 8 commits behind head on develop.

Files with missing lines Patch % Lines
vortex-vector/src/primitive/vector_mut.rs 20.00% 44 Missing ⚠️
vortex-vector/src/decimal/vector_mut.rs 13.33% 26 Missing ⚠️
vortex-vector/src/primitive/vector.rs 50.00% 17 Missing ⚠️
vortex-vector/src/decimal/vector.rs 0.00% 13 Missing ⚠️
vortex-vector/src/decimal/generic.rs 0.00% 2 Missing ⚠️
vortex-vector/src/struct_/vector.rs 33.33% 2 Missing ⚠️
vortex-vector/src/bool/vector.rs 50.00% 1 Missing ⚠️
vortex-vector/src/primitive/generic.rs 50.00% 1 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.

@gatesn gatesn merged commit 72bd594 into develop Nov 5, 2025
41 checks passed
@gatesn gatesn deleted the ct/vv-use-self branch November 5, 2025 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants