Skip to content

Commit

Permalink
Printout SHA for review
Browse files Browse the repository at this point in the history
Signed-off-by: Alex Ellis (OpenFaaS Ltd) <[email protected]>
  • Loading branch information
alexellis committed Jan 16, 2020
1 parent 2404c53 commit e291cda
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion handler/comment_handler.go
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ func HandleComment(req types.IssueCommentOuter, config config.Config, derekConfi
feedback, err = merger.Merge(req, command.Type, command.Value)

if len(feedback) > 0 {
log.Printf("Feedback: %s", feedback)
log.Printf("Feedback: %s\n", feedback)
}

if err != nil {
Expand Down
1 change: 1 addition & 0 deletions handler/merge.go
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ func (m *merge) Merge(req types.IssueCommentOuter, cmdType string, cmdValue stri

mustApproveConfirmed := []github.PullRequestReview{}
for _, r := range reviews {
fmt.Printf("Review state: %s, commitID: %s, HEADSHA: %s\n", r.GetState(), r.GetCommitID(), pr.GetHead().GetSHA())
for _, approver := range mustApprove {
if r.GetState() == "APPROVED" &&
r.GetUser().GetLogin() == approver &&
Expand Down

0 comments on commit e291cda

Please sign in to comment.