Skip to content

Support sort(keys(::Dict)) and sort(values(::Dict)) #56978

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

Merged
merged 6 commits into from
Mar 19, 2025

Conversation

LilithHafner
Copy link
Member

Part of the sorting iterables saga:
#38328
#46104
#51977
#52010
#54494

@LilithHafner LilithHafner added iteration Involves iteration or the iteration protocol sorting Put things in order labels Jan 6, 2025
@nsajko nsajko added the collections Data structures holding multiple items, e.g. sets label Jan 7, 2025
Co-authored-by: Neven Sajko <4944410+nsajko@users.noreply.github.com>
@LilithHafner LilithHafner merged commit be3221f into master Mar 19, 2025
5 of 7 checks passed
@LilithHafner LilithHafner deleted the lh/sort-key-value branch March 19, 2025 14:54
LilithHafner added a commit that referenced this pull request Mar 26, 2025
In cases where a function is documented as 
```
    function(arg::ArgT, arg2::Arg2T) -> RetT

...
```
I either switched ` -> ` to `::` or switched `RetT` to `ret_name::RetT`.

From the recommendation and justification from @nsajko here:
#56978 (comment) and
applied throughout the repo.

As documented here #57583

Also includes some minor changes to touched lines (e.g. removing annotations that are just clutter)
serenity4 pushed a commit to serenity4/julia that referenced this pull request May 1, 2025
In cases where a function is documented as 
```
    function(arg::ArgT, arg2::Arg2T) -> RetT

...
```
I either switched ` -> ` to `::` or switched `RetT` to `ret_name::RetT`.

From the recommendation and justification from @nsajko here:
JuliaLang#56978 (comment) and
applied throughout the repo.

As documented here JuliaLang#57583

Also includes some minor changes to touched lines (e.g. removing annotations that are just clutter)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
collections Data structures holding multiple items, e.g. sets iteration Involves iteration or the iteration protocol sorting Put things in order
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants