mirror of
https://github.com/git-chglog/git-chglog.git
synced 2026-01-22 18:06:11 +00:00
BREAKING CHANGE: `JiraIssueId` has been renamed to `JiraIssueID`. This impacts the value for `pattern_maps` in `config.yml`. * chore(ci): add golangci-lint action * chore(lint): address errcheck lint failures * chore(lint): address misspell lint failures * chore(lint): address gocritic lint failures * chore(lint): address golint lint failures * chore(lint): address structcheck lint failures * chore(lint): address gosimple lint failures * chore(lint): address gofmt lint failures * chore(ci): port to official golangci-lint github action * Update golangci configuration for better coverage Signed-off-by: Khosrow Moossavi <khos2ow@gmail.com> * fix: file is not goimports-ed Signed-off-by: Khosrow Moossavi <khos2ow@gmail.com> * fix: golint and exported functions comments Signed-off-by: Khosrow Moossavi <khos2ow@gmail.com> * chore(lint): address gosec G304 warning * chore(lint): address uparam warnings * chore(lint): address scopelint lint failures * fix: cyclomatic complexity Signed-off-by: Khosrow Moossavi <khos2ow@gmail.com> * chore(lint): address prealloc warning, noting that we are warning for now * chore(lint): address govet and errorlint failures * chore: clean up defer logic when checking errors Co-authored-by: Khosrow Moossavi <khos2ow@gmail.com>
64 lines
1 KiB
Go
64 lines
1 KiB
Go
package chglog
|
|
|
|
import (
|
|
"strings"
|
|
)
|
|
|
|
func commitFilter(commits []*Commit, filters map[string][]string, noCaseSensitive bool) []*Commit { //nolint:gocyclo
|
|
// NOTE(khos2ow): this function is over our cyclomatic complexity goal.
|
|
// Be wary when adding branches, and look for functionality that could
|
|
// be reasonably moved into an injected dependency.
|
|
|
|
res := []*Commit{}
|
|
|
|
for _, commit := range commits {
|
|
include := false
|
|
|
|
if len(filters) == 0 {
|
|
include = true
|
|
}
|
|
|
|
for key, values := range filters {
|
|
prop, ok := dotGet(commit, key)
|
|
if !ok {
|
|
include = false
|
|
break
|
|
}
|
|
|
|
str, ok := prop.(string)
|
|
if !ok {
|
|
include = false
|
|
break
|
|
}
|
|
|
|
if noCaseSensitive {
|
|
str = strings.ToLower(str)
|
|
}
|
|
|
|
exist := false
|
|
|
|
for _, val := range values {
|
|
if noCaseSensitive {
|
|
val = strings.ToLower(val)
|
|
}
|
|
|
|
if str == val {
|
|
exist = true
|
|
}
|
|
}
|
|
|
|
if !exist {
|
|
include = false
|
|
break
|
|
}
|
|
|
|
include = true
|
|
}
|
|
|
|
if include {
|
|
res = append(res, commit)
|
|
}
|
|
}
|
|
|
|
return res
|
|
}
|