-
Notifications
You must be signed in to change notification settings - Fork 604
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: std/data_structures
#3734
Conversation
Any suggestion, objection, bikeshedding, etc from the community is welcome. |
There seems to be no concern. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I've corrected the removal version. |
Let me try some bike shedding. How about singular form: @KyleJune Do you have any other/better ideas for this new submodule name? |
I think |
Of the two, my preference would be std/data_structures to make it clear there are many. std/collections is also plural. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Move `BinaryHeap` in std/collections to std/data_structures ref: denoland/std#3734
This change moves the unstable
BinaryHeap()
,BinarySearchTree()
,ascend()
,descend()
andRedBlackTree()
APIs to the newstd/data_structures
sub-module. After a discussion with @kt3k, we decided this location for these APIs makes the most sense. This will also open up opportunities for more data-structure-related APIs in the future.The unstable files within
std/collections
were deleted, as they were... unstable. I've updated and cleaned up the previous deprecation documentation and added the sub-module to CI processes, etc.