[skywalking-eyes] branch skip-binary created (now d951e97)

[ke...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

kezhenxu94 pushed a change to branch skip-binary
in repository https://gitbox.apache.org/repos/asf/skywalking-eyes.git.


      at d951e97  Enhance `check` command to automatically skip non-text (binary) files

This branch includes the following new commits:

     new d951e97  Enhance `check` command to automatically skip non-text (binary) files

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[skywalking-eyes] 01/01: Enhance `check` command to automatically skip non-text (binary) files

[ke...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

kezhenxu94 pushed a commit to branch skip-binary
in repository https://gitbox.apache.org/repos/asf/skywalking-eyes.git

commit d951e97ec128ca4cf0f259ad77fccb6fd0193e93
Author: kezhenxu94 <ke...@apache.org>
AuthorDate: Tue Dec 22 23:04:00 2020 +0800

    Enhance `check` command to automatically skip non-text (binary) files
---
 .licenserc.yaml                 |  2 --
 license-eye/pkg/header/check.go | 11 +++++++++++
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/.licenserc.yaml b/.licenserc.yaml
index dc3b932..e4494c7 100644
--- a/.licenserc.yaml
+++ b/.licenserc.yaml
@@ -61,9 +61,7 @@ header: # `header` section is configurations for source codes license header.
   paths-ignore: # `paths-ignore` are the path list that will be ignored by license-eye.
     - 'dist'
     - 'licenses'
-    - '**/bin/**'
     - '**/*.md'
-    - '**/.DS_Store'
     - '**/testdata/**'
     - '**/go.mod'
     - '**/go.sum'
diff --git a/license-eye/pkg/header/check.go b/license-eye/pkg/header/check.go
index 5325692..8053cd3 100644
--- a/license-eye/pkg/header/check.go
+++ b/license-eye/pkg/header/check.go
@@ -19,6 +19,8 @@ package header
 
 import (
 	"bufio"
+	"io/ioutil"
+	"net/http"
 	"os"
 	"path/filepath"
 	"regexp"
@@ -122,6 +124,15 @@ func CheckFile(file string, config *ConfigHeader, result *Result) error {
 
 	var lines []string
 
+	bs, err := ioutil.ReadFile(file)
+	if err != nil {
+		return err
+	}
+	if t := http.DetectContentType(bs); !strings.HasPrefix(t, "text/") {
+		logger.Log.Debugln("Ignoring file:", file, "; type:", t)
+		return nil
+	}
+
 	scanner := bufio.NewScanner(reader)
 	for scanner.Scan() {
 		line := strings.ToLower(Punctuations.ReplaceAllString(scanner.Text(), " "))