ci: add golangci

.github/workflows/golangci-lint.yml

@@ -9,13 +9,11 @@ on:
   pull_request:
 permissions:
   contents: read
-  # Optional: allow read access to pull request. Use with `only-new-issues` option.
-  # pull-requests: read
 jobs:
   golangci:
     name: lint
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v2
+        uses: golangci/golangci-lint-action@v2

bot/adapters/storage/memorystorage.go

@@ -8,6 +8,7 @@ import (
 	"sort"
 
 	"github.com/tal-tech/go-zero/core/lang"
+	"github.com/tal-tech/go-zero/core/logx"
 	"github.com/tal-tech/go-zero/core/mr"
 	"github.com/wangbin/jiebago"
 	"github.com/wangbin/jiebago/analyse"
@@ -43,11 +44,11 @@ type (
 
 func RestoreMemoryStorage(decoder *gob.Decoder) (*memoryStorage, error) {
 	var segmenter jiebago.Segmenter
-	segmenter.LoadDictionary(dictFile)
+	logx.Must(segmenter.LoadDictionary(dictFile))
 	var extracter analyse.TagExtracter
-	extracter.LoadDictionary(dictFile)
-	extracter.LoadIdf(idfFile)
-	extracter.LoadStopWords(stopWordsFile)
+	logx.Must(extracter.LoadDictionary(dictFile))
+	logx.Must(extracter.LoadIdf(idfFile))
+	logx.Must(extracter.LoadStopWords(stopWordsFile))
 
 	var keys []string
 	responses := make(map[string]map[string]int)
@@ -76,11 +77,11 @@ func RestoreMemoryStorage(decoder *gob.Decoder) (*memoryStorage, error) {
 
 func NewMemoryStorage() *memoryStorage {
 	var segmenter jiebago.Segmenter
-	segmenter.LoadDictionary(dictFile)
+	logx.Must(segmenter.LoadDictionary(dictFile))
 	var extracter analyse.TagExtracter
-	extracter.LoadDictionary(dictFile)
-	extracter.LoadIdf(idfFile)
-	extracter.LoadStopWords(stopWordsFile)
+	logx.Must(extracter.LoadDictionary(dictFile))
+	logx.Must(extracter.LoadIdf(idfFile))
+	logx.Must(extracter.LoadStopWords(stopWordsFile))
 
 	return &memoryStorage{
 		segmenter: &segmenter,
@@ -382,23 +383,3 @@ func splitStrings(slice []string, size int) []*keyChunk {
 
 	return result
 }
-
-func (storage *memoryStorage) print() {
-	fmt.Println("Questions:")
-	for _, key := range storage.keys {
-		fmt.Printf("\t%s\n", key)
-	}
-
-	fmt.Println("Responses:")
-	for key, value := range storage.responses {
-		fmt.Printf("\t%s:\n", key)
-		for text, occurrence := range value {
-			fmt.Printf("\t\t%s:\t%d\n", text, occurrence)
-		}
-	}
-
-	fmt.Println("Indexes:")
-	for key, ids := range storage.indexes {
-		fmt.Printf("\t%s: %v\n", key, ids)
-	}
-}

bot/nlp/comparisons.go

@@ -106,10 +106,3 @@ func min(a int, b int) int {
 	}
 	return a
 }
-
-func max(a int, b int) int {
-	if b > a {
-		return b
-	}
-	return a
-}

bot/trainers.go

@@ -86,7 +86,9 @@ func (trainer *CorpusTrainer) Train(data interface{}) error {
 
 	for _, convs := range corpora {
 		for _, conv := range convs {
-			convTrainer.Train(conv)
+			if err := convTrainer.Train(conv); err != nil {
+				return err
+			}
 		}
 	}