Welcome to mirror list, hosted at ThFree Co, Russian Federation.

github.com/twbs/rorschach.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat (limited to 'src/main/scala/com/getbootstrap/rorschach/github/GitHubPullRequestCommenter.scala')
-rw-r--r--src/main/scala/com/getbootstrap/rorschach/github/GitHubPullRequestCommenter.scala12
1 files changed, 7 insertions, 5 deletions
diff --git a/src/main/scala/com/getbootstrap/rorschach/github/GitHubPullRequestCommenter.scala b/src/main/scala/com/getbootstrap/rorschach/github/GitHubPullRequestCommenter.scala
index 77171c2..7d394c6 100644
--- a/src/main/scala/com/getbootstrap/rorschach/github/GitHubPullRequestCommenter.scala
+++ b/src/main/scala/com/getbootstrap/rorschach/github/GitHubPullRequestCommenter.scala
@@ -31,7 +31,7 @@ class GitHubPullRequestCommenter extends GitHubActorWithLogging {
}
override def receive = {
- case PullRequestFeedback(prNum, requester, messages) => {
+ case PullRequestFeedback(repo, prNum, requester, messages) => {
val username = requester.getLogin
val messagesMarkdown = messages.map{ "* " + _ }.mkString("\n")
val commentMarkdown = s"""
@@ -47,14 +47,16 @@ class GitHubPullRequestCommenter extends GitHubActorWithLogging {
|(*Please note that this is a [fully automated](https://github.com/cvrebert/rorschach) comment.*)
""".stripMargin
- tryToCommentOn(BootstrapRepoId, prNum, commentMarkdown) match {
+ tryToCommentOn(repo, prNum, commentMarkdown) match {
case Success(comment) => log.info(s"Successfully posted comment ${comment.getUrl} for ${prNum}")
case Failure(exc) => log.error(exc, s"Error posting comment for ${prNum}")
}
- tryToClose(BootstrapRepoId, prNum) match {
- case Success(_) => log.info(s"Successfully closed ${prNum} due to failed audit(s)")
- case Failure(exc) => log.error(exc, s"Error closing ${prNum}")
+ if (settings.CloseBadPullRequests) {
+ tryToClose(repo, prNum) match {
+ case Success(_) => log.info(s"Successfully closed ${prNum} due to failed audit(s)")
+ case Failure(exc) => log.error(exc, s"Error closing ${prNum}")
+ }
}
}
}