Skip to content

Create testfile#4

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

Create testfile#4
rmetzger wants to merge 1 commit into
flinkbot:masterfrom
rmetzger:patch-1

Conversation

@rmetzger

Copy link
Copy Markdown

No description provided.

@flinkbot

flinkbot commented Jan 23, 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.

@rmetzger

Copy link
Copy Markdown
Author

@flinkbot approve contribution

@dawidwys

Copy link
Copy Markdown

@flinkbot approve consensus

@uce

uce commented Jan 23, 2019

Copy link
Copy Markdown

@flinkbot approve quality

@cewood

cewood commented Jan 23, 2019

Copy link
Copy Markdown

@flinkbot what does it mean to live

@dawidwys

Copy link
Copy Markdown

@flinkbot attention @dawidwys

@rmetzger

Copy link
Copy Markdown
Author

I hope I will be less nervous about the bot's functionality at some point :)
Right now, every comment here reduces my life expectancy by 5 minutes, because of the stress.

@uce

uce commented Jan 23, 2019

Copy link
Copy Markdown

Haha. It seems to work fine. How do you run it?

@rmetzger

Copy link
Copy Markdown
Author

On a micro instance in google cloud $5 a month or so :)

@rmetzger

Copy link
Copy Markdown
Author

@flinkbot disapprove contribution

@dawidwys

Copy link
Copy Markdown

@flinkbot approve quality architecture

@rmetzger

Copy link
Copy Markdown
Author

Sorry to disappoint you :)

@fhueske

fhueske commented Jan 28, 2019

Copy link
Copy Markdown

@flinkbot approve architecture

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.

6 participants