Skip to content

[NFC] Formatting cleanup to help with next conflicts #39770

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 1 commit into from
Oct 15, 2021

Conversation

bnbarham
Copy link
Contributor

No description provided.

@bnbarham
Copy link
Contributor Author

I've been going through the main -> next merge conflicts and there were some where the formatting made a little more sense in next, so that's these.

I'm not too sure about the SmallVector changes, happy to leave them out if anyone thinks the main ones make more sense.

I could also fix all the conflicts as is and just pick all the main changes. Let me know your thoughts!

Copy link
Member

@etcwilde etcwilde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for digging into these. It's been on my todo list. :)

@etcwilde
Copy link
Member

@swift-ci please test

@etcwilde
Copy link
Member

The Windows failure is just that the bot is out of space. I'm going to go ahead and merge this so that I can continue cleaning up the remaining merge conflicts. Thanks!

@etcwilde etcwilde merged commit 3676977 into swiftlang:main Oct 15, 2021
@bnbarham bnbarham deleted the main-to-next-conflicts branch October 15, 2021 22:43
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