Skip to content

Update README.md #5

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.md #5

wants to merge 1 commit into from

Conversation

fhueske
Copy link

@fhueske fhueske commented Jan 28, 2019

Checking how the bot handles new PRs. 😎

@flinkbot
Copy link
Owner

flinkbot commented Jan 28, 2019

Thanks a lot for your contribution to the Apache Flink project. I'm the @flinkbot. I help the community
to review your pull request. We will use this comment to track the progress of the review.

Review Progress

  • 1. The [contribution] is well-described.
  • 2. There is [consensus] that the contribution should go into to Flink.
  • 3. [Does not need specific [attention] | Needs specific attention for X | Has attention for X by Y]
  • 4. The [architecture] is sound.
  • 5. Overall code [quality] is good.

Please see the Pull Request Review Guide if you have questions about the review process.

@fhueske
Copy link
Author

fhueske commented Jan 28, 2019

@flinkbot approve contribution
@flinkbot approve consensus

@rmetzger
Copy link

damn

@rmetzger
Copy link

Looks like I need to do some debugging. I had a similar issue before

@klion26
Copy link

klion26 commented Jan 29, 2019

@flinkbot approve contribution

@rmetzger
Copy link

@fhueske It seem to have worked? Maybe with a delay

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.

4 participants