Skip to content

Update README.md#5

Open
fhueske wants to merge 1 commit into
flinkbot:masterfrom
fhueske:patch-1
Open

Update README.md#5
fhueske wants to merge 1 commit into
flinkbot:masterfrom
fhueske:patch-1

Conversation

@fhueske

@fhueske fhueske commented Jan 28, 2019

Copy link
Copy Markdown

Checking how the bot handles new PRs. 😎

@flinkbot

flinkbot commented Jan 28, 2019

Copy link
Copy Markdown
Owner

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

fhueske commented Jan 28, 2019

Copy link
Copy Markdown
Author

@flinkbot approve contribution
@flinkbot approve consensus

@rmetzger

Copy link
Copy Markdown

damn

@rmetzger

Copy link
Copy Markdown

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

@klion26

klion26 commented Jan 29, 2019

Copy link
Copy Markdown

@flinkbot approve contribution

@rmetzger

Copy link
Copy Markdown

@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