search

databricks / spark-pr-dashboard

Dashboard to aid in Spark pull request reviews
spark-prs.appspot.com
Apache License 2.0
54 stars 46 forks source link

Allow merging pull requests directly in the pr dashboard #7

Closed rxin closed 10 years ago

rxin commented 10 years ago

This is pretty hard since it'd require authentication.

JoshRosen commented 10 years ago

This would actually be pretty tricky, since we'd want to capture all of the functionality of the Spark PR merge script. Also, I don't know how easy this would be to do on AppEngine, where we won't be able to run arbitrary programs like git.

I don't think that actually running the merge script is a bottleneck and it's probably better to have a human in the loop that can look at the git commit before pushing, etc.