Skip to content

feat(data-structures): stabilize IndexedHeap#7245

Open
tomas-zijdemans wants to merge 1 commit into
denoland:mainfrom
tomas-zijdemans:stabilize-indexed-heap
Open

feat(data-structures): stabilize IndexedHeap#7245
tomas-zijdemans wants to merge 1 commit into
denoland:mainfrom
tomas-zijdemans:stabilize-indexed-heap

Conversation

@tomas-zijdemans

@tomas-zijdemans tomas-zijdemans commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Promotes IndexedHeap from unstable to stable. The API design was already accepted during the unstable period, so this is the mechanical part:

  • Renamed unstable_indexed_heap.ts to indexed_heap.ts (test file too)
  • Export path is now ./indexed-heap, and mod.ts re-exports the module
  • Dropped the 21 @experimental tags and pointed all 29 JSDoc example imports at the new path

One small change beyond the rename: the NaN rejection error now names the operation that was attempted. push() and the constructor throw Cannot push into IndexedHeap: priority is NaN, set() throws Cannot set priority in IndexedHeap: priority is NaN. Before, all paths said "Cannot set priority".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant