Skip to content

Conversation

alpinegizmo
Copy link
Contributor

Reading through the docs, I found some small problems -- typos, broken links, awkward phrases. (This was originally submitted as PR 2519, but I'd forgotten to base that pull request on a feature branch, so I decided to start over.)

Copy link
Contributor

@aljoscha aljoscha left a comment

Choose a reason for hiding this comment

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

Changes look very good. I'm merging 👍

@greghogan
Copy link
Contributor

Oh no! I just committed.

@asfgit asfgit closed this in 7212202 Sep 21, 2016
@aljoscha
Copy link
Contributor

No worries. 😃

@greghogan
Copy link
Contributor

Thank you for the contribution @alpinegizmo.

liuyuzhong pushed a commit to liuyuzhong/flink that referenced this pull request Dec 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants