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

fix documentation comment of flatMap<SegmentOfResult : Sequence> #15845

Merged
merged 1 commit into from Apr 10, 2018

Conversation

Projects
None yet
3 participants
@creaaa
Contributor

creaaa commented Apr 10, 2018

Hello, I'm Masa, Swift lover.

I noticed that one documentation comment remained old and doesn't change since 2 years ago, so I fixed it.

Is this PR acceptable?
This PR only includes a fixed single file.
I really appreciate it if you would tell me if there is a defect in this PR.

I'm deeply grateful to you for all your passionate work.

Thank you,
Masa

@harlanhaskins

This comment has been minimized.

Collaborator

harlanhaskins commented Apr 10, 2018

Good catch, @creaaa! This looks good to me, but I want @natecook1000 to spot-check it if he has time.

@harlanhaskins

This comment has been minimized.

Collaborator

harlanhaskins commented Apr 10, 2018

@swift-ci please smoke test

@natecook1000

This comment has been minimized.

Member

natecook1000 commented Apr 10, 2018

@creaaa Thanks so much!

@natecook1000 natecook1000 merged commit d8e0585 into apple:master Apr 10, 2018

2 checks passed

Swift Test Linux Platform (smoke test)
Details
Swift Test OS X Platform (smoke test)
Details
@creaaa

This comment has been minimized.

Contributor

creaaa commented Apr 10, 2018

Thanks too, @harlanhaskins, @natecook1000 !
This made my love to Swift unwavering. I'll keep learning to contribute more!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment