Skip to content

Conversation

@simonjbeaumont
Copy link
Contributor

The following paragraph appears twice in the documentation comments for FileDescriptor.writeAlll(_:):

/// This method either writes the entire contents of `sequence`,
/// or throws an error if only part of the content was written.

This PR cleans that up.

@simonjbeaumont simonjbeaumont requested a review from lorentey May 6, 2022 11:44
Copy link
Contributor

@milseman milseman left a comment

Choose a reason for hiding this comment

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

LGTM

@milseman
Copy link
Contributor

milseman commented May 6, 2022

@swift-ci please test

@simonjbeaumont
Copy link
Contributor Author

@milseman OK to merge this one? For my own education, what's the merge process in this repo—once the PR is approved, should the author merge or one of the maintainers?

@milseman
Copy link
Contributor

I can merge it. I don't know if you have merge or CI permissions

@milseman milseman merged commit fea7d50 into apple:main May 16, 2022
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