From 642d8f12cbb40e75794028fac8a41088e624d5fe Mon Sep 17 00:00:00 2001
From: cagiti <caicooper82@gmail.com>
Date: Tue, 17 Nov 2020 12:07:36 +0000
Subject: [PATCH 1/2] fix: add missing test case where hosts.yml doesn't exist

---
 go.sum                 |  2 ++
 pkg/api/config.go      |  3 ++-
 pkg/api/config_test.go | 20 ++++++++++++++++++++
 3 files changed, 24 insertions(+), 1 deletion(-)

diff --git a/go.sum b/go.sum
index f114929..722c942 100644
--- a/go.sum
+++ b/go.sum
@@ -607,11 +607,13 @@ k8s.io/apimachinery v0.19.0 h1:gjKnAda/HZp5k4xQYjL0K/Yb66IvNqjthCb03QlKpaQ=
 k8s.io/apimachinery v0.19.0/go.mod h1:DnPGDnARWFvYa3pMHgSxtbZb7gpzzAZ1pTfaUNDVlmA=
 k8s.io/apimachinery v0.19.3 h1:bpIQXlKjB4cB/oNpnNnV+BybGPR7iP5oYpsOTEJ4hgc=
 k8s.io/apimachinery v0.19.3/go.mod h1:DnPGDnARWFvYa3pMHgSxtbZb7gpzzAZ1pTfaUNDVlmA=
+k8s.io/apimachinery v0.19.4 h1:+ZoddM7nbzrDCp0T3SWnyxqf8cbWPT2fkZImoyvHUG0=
 k8s.io/apimachinery v0.19.4/go.mod h1:DnPGDnARWFvYa3pMHgSxtbZb7gpzzAZ1pTfaUNDVlmA=
 k8s.io/client-go v0.19.0 h1:1+0E0zfWFIWeyRhQYWzimJOyAk2UT7TiARaLNwJCf7k=
 k8s.io/client-go v0.19.0/go.mod h1:H9E/VT95blcFQnlyShFgnFT9ZnJOAceiUHM3MlRC+mU=
 k8s.io/client-go v0.19.3 h1:ctqR1nQ52NUs6LpI0w+a5U+xjYwflFwA13OJKcicMxg=
 k8s.io/client-go v0.19.3/go.mod h1:+eEMktZM+MG0KO+PTkci8xnbCZHvj9TqR6Q1XDUIJOM=
+k8s.io/client-go v0.19.4 h1:85D3mDNoLF+xqpyE9Dh/OtrJDyJrSRKkHmDXIbEzer8=
 k8s.io/client-go v0.19.4/go.mod h1:ZrEy7+wj9PjH5VMBCuu/BDlvtUAku0oVFk4MmnW9mWA=
 k8s.io/gengo v0.0.0-20200413195148-3a45101e95ac/go.mod h1:ezvh/TsK7cY6rbqRK0oQQ8IAqLxYwwyPxAX1Pzy0ii0=
 k8s.io/klog/v2 v2.0.0/go.mod h1:PBfzABfn139FHAV07az/IF9Wp1bkk3vpT2XSJ76fSDE=
diff --git a/pkg/api/config.go b/pkg/api/config.go
index 7a6b6f8..a8b5fe3 100644
--- a/pkg/api/config.go
+++ b/pkg/api/config.go
@@ -4,6 +4,7 @@ import (
 	"io/ioutil"
 	"os"
 	"path"
+	"strings"
 
 	"github.com/ghodss/yaml"
 	"github.com/plumming/dx/pkg/util"
@@ -22,7 +23,7 @@ func ParseDefaultConfig(cf, hf string) (Config, error) {
 	// ~/.config/gh/hosts.yml lets try and load from
 	// there initially
 	config, err := parseHostsFile(hf)
-	if err != nil {
+	if err != nil && !strings.Contains(err.Error(), "no such file or directory") {
 		return nil, err
 	}
 	if config.HasHosts() {
diff --git a/pkg/api/config_test.go b/pkg/api/config_test.go
index ce5e6e5..5b3a992 100644
--- a/pkg/api/config_test.go
+++ b/pkg/api/config_test.go
@@ -228,3 +228,23 @@ func TestParseDefaultConfigWithConfigFile(t *testing.T) {
 	token := config.GetToken("github.com")
 	assert.Equal(t, "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx", token)
 }
+
+func TestParseDefaultConfigWithNoHostsFile(t *testing.T) {
+	configFile, err := ioutil.TempFile(os.TempDir(), "TestParseDefaultConfigWithNoHostsFile2")
+	if err != nil {
+		panic(err)
+	}
+	defer os.Remove(configFile.Name())
+
+	err = ioutil.WriteFile(configFile.Name(), []byte(dummyConfigFile), 0600)
+	assert.NoError(t, err)
+
+	config, err := ParseDefaultConfig(configFile.Name(), "hosts-file-does-not-exist")
+	assert.NoError(t, err)
+
+	user := config.GetUser("github.com")
+	assert.Equal(t, "testuser", user)
+
+	token := config.GetToken("github.com")
+	assert.Equal(t, "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx", token)
+}

From 3616a2d59f177e940992dbde303d02245d10d7c4 Mon Sep 17 00:00:00 2001
From: cagiti <caicooper82@gmail.com>
Date: Tue, 17 Nov 2020 12:15:49 +0000
Subject: [PATCH 2/2] fix: update version of golangci lint action

---
 .github/workflows/golangci-lint.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/golangci-lint.yml b/.github/workflows/golangci-lint.yml
index 83d1aeb..80bc2d4 100644
--- a/.github/workflows/golangci-lint.yml
+++ b/.github/workflows/golangci-lint.yml
@@ -13,7 +13,7 @@ jobs:
     steps:
       - uses: actions/checkout@v2
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v1
+        uses: golangci/golangci-lint-action@v2
         with:
           # Required: the version of golangci-lint is required and must be specified without patch version: we always use the latest patch version.
-          version: v1.27
+          version: v1.28