Skip to content

Create testfile #4

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

Create testfile #4

wants to merge 1 commit into from

Conversation

rmetzger
Copy link

No description provided.

@flinkbot
Copy link
Owner

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

@rmetzger
Copy link
Author

@flinkbot approve contribution

@dawidwys
Copy link

@flinkbot approve consensus

@uce
Copy link

uce commented Jan 23, 2019

@flinkbot approve quality

@cewood
Copy link

cewood commented Jan 23, 2019

@flinkbot what does it mean to live

@dawidwys
Copy link

@flinkbot attention @dawidwys

@rmetzger
Copy link
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
Copy link

uce commented Jan 23, 2019

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

@rmetzger
Copy link
Author

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

@rmetzger
Copy link
Author

@flinkbot disapprove contribution

@dawidwys
Copy link

@flinkbot approve quality architecture

@rmetzger
Copy link
Author

Sorry to disappoint you :)

@fhueske
Copy link

fhueske commented Jan 28, 2019

@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