Changelog Overhaul #18

Merged
lafriks merged 10 commits from jolheiser/changelog:master into master 2020-01-09 07:43:22 +00:00
3 changed files with 6 additions and 4 deletions
Showing only changes of commit e84ee40887 - Show all commits

View File

@ -44,8 +44,10 @@ func LoadConfig() (*Config, error) {
return nil, err
}
if config.SkipRegex, err = regexp.Compile(config.SkipLabels); err != nil {
return nil, err
if len(config.SkipLabels) > 0 {
if config.SkipRegex, err = regexp.Compile(config.SkipLabels); err != nil {
return nil, err
}
}
return config, nil

View File

@ -66,7 +66,7 @@ func runGenerate(cmd *cli.Context) error {
for _, pr := range result.Issues {
var label string
for _, lb := range pr.Labels {
if config.SkipRegex.MatchString(lb.GetName()) {
if config.SkipRegex != nil && config.SkipRegex.MatchString(lb.GetName()) {
continue PRLoop
}

View File

@ -49,6 +49,6 @@ func main() {
err := app.Run(os.Args)
if err != nil {
fmt.Printf("Failed to run app with %s: %v", os.Args[1:], err)
fmt.Printf("Failed to run app with %s: %v\n", os.Args[1:], err)
}
}