Skip to content
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

Trivial typo in docker-flink/README.md #1463

Closed

Conversation

petervandenabeele
Copy link
Contributor

@fhueske
Copy link
Contributor

fhueske commented Dec 17, 2015

Hi @petervandenabeele, thanks for the patch!

+1 to merge

@petervandenabeele
Copy link
Contributor Author

Hmmm, one of the Travis tests failed

  • I presume this is a flickering test from the context of other PR tests passing around that time
  • my patch is only a small documentation change

Will it be merged as is, or should I try to trigger a new Travis test by pushing an essentially empty commit in this PR ?

@zentol
Copy link
Contributor

zentol commented Dec 17, 2015

yes it's not related to your change (there are a bunch of unstable tests), it will be merged and you don't have to do anything.

@mxm
Copy link
Contributor

mxm commented Dec 17, 2015

Thanks for the pull request!

@asfgit asfgit closed this in d6e9fa0 Dec 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants