Skip to content

[Index] Write stdlib records in alphabetical order #67954

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

Conversation

bnbarham
Copy link
Contributor

The order for writing records of the stdlib currently depends on StringMap iteration (in a slightly roundabout manner). Sort these alphabetically instead.

@bnbarham
Copy link
Contributor Author

@swift-ci please test

@bnbarham
Copy link
Contributor Author

@swift-ci please test Linux platform

@bnbarham
Copy link
Contributor Author

swiftlang/sourcekit-lsp#800

@swift-ci please test Linux platform

@bnbarham
Copy link
Contributor Author

swiftlang/sourcekit-lsp#801

@swift-ci please test

The order for writing records of the stdlib currently depends on
`StringMap` iteration (in a slightly roundabout manner). Sort these
alphabetically instead.
@bnbarham bnbarham force-pushed the do-not-depend-on-stringmap-iteration branch from 4fb2a22 to 5cdc7db Compare August 19, 2023 02:38
@bnbarham
Copy link
Contributor Author

@swift-ci please test

@bnbarham bnbarham merged commit ec485ba into swiftlang:main Aug 20, 2023
@bnbarham bnbarham deleted the do-not-depend-on-stringmap-iteration branch August 20, 2023 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants