Browse Source

Avoid 0 new commits messages to be send (#11082)

Fixes #10498
tags/v1.13.0-dev
Cornel GitHub 5 years ago
parent
commit
f3b5339b77
1 changed files with 1 additions and 7 deletions
  1. +1
    -7
      modules/repofiles/action.go

+ 1
- 7
modules/repofiles/action.go View File

@@ -249,12 +249,12 @@ func CommitRepoAction(optsList ...*CommitRepoActionOptions) error {
IsPrivate: repo.IsPrivate, IsPrivate: repo.IsPrivate,
} }


var isHookEventPush = true
switch opType { switch opType {
case models.ActionCommitRepo: // Push case models.ActionCommitRepo: // Push
if opts.IsNewBranch() { if opts.IsNewBranch() {
notification.NotifyCreateRef(pusher, repo, "branch", opts.RefFullName) notification.NotifyCreateRef(pusher, repo, "branch", opts.RefFullName)
} }
notification.NotifyPushCommits(pusher, repo, opts.RefFullName, opts.OldCommitID, opts.NewCommitID, opts.Commits)
case models.ActionDeleteBranch: // Delete Branch case models.ActionDeleteBranch: // Delete Branch
notification.NotifyDeleteRef(pusher, repo, "branch", opts.RefFullName) notification.NotifyDeleteRef(pusher, repo, "branch", opts.RefFullName)


@@ -263,12 +263,6 @@ func CommitRepoAction(optsList ...*CommitRepoActionOptions) error {


case models.ActionDeleteTag: // Delete Tag case models.ActionDeleteTag: // Delete Tag
notification.NotifyDeleteRef(pusher, repo, "tag", opts.RefFullName) notification.NotifyDeleteRef(pusher, repo, "tag", opts.RefFullName)
default:
isHookEventPush = false
}

if isHookEventPush {
notification.NotifyPushCommits(pusher, repo, opts.RefFullName, opts.OldCommitID, opts.NewCommitID, opts.Commits)
} }
} }




Loading…
Cancel
Save