No comment if multiple sources are given

[int128]

After #955, the controller does not create a comment to a pull request if multiple sources are given. For example, int128/argocd-commenter-e2e-test#2531.

It needs to consider Sources here:

argocd-commenter/internal/notification/pull.go

Lines 21 to 73 in 47dacc1

	func isPullRequestRelatedToEvent(pull github.PullRequest, app argocdv1alpha1.Application) bool {
	if app.Spec.Source == nil {
	return false
	}
	// support manifest path annotation
	// see https://argo-cd.readthedocs.io/en/stable/operator-manual/high_availability/#webhook-and-manifest-paths-annotation
	// https://github.com/int128/argocd-commenter/pull/656
	manifestGeneratePaths := getManifestGeneratePaths(app)
	for _, file := range pull.Files {
	if strings.HasPrefix(file, app.Spec.Source.Path) {
	return true
	}
	for _, path := range manifestGeneratePaths {
	if strings.HasPrefix(file, path) {
	return true
	}
	}
	}
	return false
	}
	// getManifestGeneratePaths returns canonical paths of "argocd.argoproj.io/manifest-generate-paths" annotation.
	// It returns nil if the field is nil or empty.
	// https://argo-cd.readthedocs.io/en/stable/operator-manual/high_availability/#webhook-and-manifest-paths-annotation
	func getManifestGeneratePaths(app argocdv1alpha1.Application) []string {
	if app.Annotations == nil {
	return nil
	}
	if app.Spec.Source == nil {
	return nil
	}
	var canonicalPaths []string
	annotatedPaths := strings.Split(app.Annotations["argocd.argoproj.io/manifest-generate-paths"], ";")
	for _, path := range annotatedPaths {
	if path == "" {
	return nil
	}
	// convert to absolute path
	absolutePath := path
	if !filepath.IsAbs(path) {
	absolutePath = filepath.Join(app.Spec.Source.Path, path)
	}
	// remove leading slash
	if absolutePath[0:1] == "/" {
	absolutePath = absolutePath[1:]
	}
	// add to list of manifest paths
	canonicalPaths = append(canonicalPaths, absolutePath)
	}
	return canonicalPaths
	}