Update the collection guide for 2.13 #1227
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
julienrf
force-pushed
the
new-collections-guide
branch
from
06eb802 to
22e302c
Compare
|
Could you review, please, @szeiger? |
(we need to finish 2.13.0-RC1 first, then we'll give higher priority to doc work) |
|
note to self, I should go ahead and give this a light round of review and then merge it. if it needs more work we can always improve it further later |
|
@julienrf would you mind fixing this up before merge? |
|
These links are computed by using the |
|
hardcoding would be fine. if you're feeling a little more ambitious, you could add a I'll submit a separate PR bumping |
Its content is copied from the current “Collections” guide.
- Remove documentation of deprecated or removed collections
(Traversable, immutable.Stack, LinkedList, DoubleLinkedList,
synchronized sets and maps), operations (toStream, copyToBuffer, hasDefiniteSize, addition and removal
operations on root collections, union, reverseMap, prefixLength) and generic package
- Update documentation of collections that have been replaced
by other collections (Stream, WrappedArray, HashMap, HashSet,
ArrayStack, View), or renamed operations (retain, transform)
- Add documentation for new collections (immutable.ArraySeq,
VectorMap, ArrayDeque) and new operations (unfold, toLazyList,
knownSize, distinct, distinctBy, groupMap, groupMapReduce, xxxInPlace,
sizeCompare, sizeIs, minOption, maxOption)
- Harmonize code style
- don’t use infix syntax with operations taking more than one param
- show alphabetic name of operations before showing their symbolic
alias
- Remove “Migrating from 2.7” page
julienrf
force-pushed
the
new-collections-guide
branch
from
2422d0e to
151c0a0
Compare
|
I felt ambitious. |
julienrf
force-pushed
the
new-collections-guide
branch
from
151c0a0 to
ff77e62
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #995
I’ve split the PR into two commits for easier review. The first commit duplicates the current collection guide into a “Scala 2.13” guide, without changing its contents. The second commit applies the necessary changes. As a result, you can review just the second commit to see the differences with the current collections guide.
Here are some screenshots of the layout and diagrams.