Skip to content

Rename Array -> DynArray#6760

Open
gatesn wants to merge 2 commits intodevelopfrom
ngates/dyn-array
Open

Rename Array -> DynArray#6760
gatesn wants to merge 2 commits intodevelopfrom
ngates/dyn-array

Conversation

@gatesn
Copy link
Contributor

@gatesn gatesn commented Mar 2, 2026

As per #6544 and the migration plan in #6735

Signed-off-by: Nicholas Gates <nick@nickgates.com>
@gatesn gatesn added the changelog/break A breaking API change label Mar 2, 2026
@gatesn gatesn enabled auto-merge (squash) March 2, 2026 21:12
Copy link
Contributor

@connortsui20 connortsui20 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

huge moment!

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 2, 2026

Merging this PR will improve performance by 12.36%

⚡ 1 improved benchmark
✅ 953 untouched benchmarks
⏩ 1466 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation bitwise_not_vortex_buffer_mut[128] 530.3 ns 471.9 ns +12.36%

Comparing ngates/dyn-array (5b0277f) with develop (85bcaaa)

Open in CodSpeed

Footnotes

  1. 1466 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

Signed-off-by: Nicholas Gates <nick@nickgates.com>
@gatesn gatesn disabled auto-merge March 2, 2026 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/break A breaking API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants