Merge pull request #11 from Nightapes/remove/draft

fix(github): remove 'draft' option because releases will always be pu…

.github/workflows/main.yml

@@ -0,0 +1,39 @@
+name: Go
+on: [push, pull_request]
+jobs:
+
+  build:
+    name: Build
+    runs-on: ubuntu-latest
+    steps:
+    
+    - name: Set up Go 1.12
+      uses: actions/setup-go@v1
+      with:
+        go-version: 1.12
+      id: go
+
+    - name: Check out code into the Go module directory
+      uses: actions/checkout@v1
+
+    - name: Lint
+      run: |
+        export PATH=$PATH:$(go env GOPATH)/bin
+        curl -sfL https://install.goreleaser.com/github.com/golangci/golangci-lint.sh | sh -s -- -b $(go env GOPATH)/bin v1.16.0
+        golangci-lint run ./...
+
+    - name: Run tests
+      run: go test ./...
+
+    - name: Build binary
+      env:
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+      run: |
+        go build -o build/go-semantic-release-temp ./cmd/go-semantic-release/
+        go build -o build/go-semantic-release  -ldflags "-w -s --X main.version=`./build/go-semantic-release-temp next`" ./cmd/go-semantic-release/
+        GOOS=windows GOARCH=386 go build -o build/go-semantic-release.exe  -ldflags "-w -s -X main.version=`./build/go-semantic-release-temp next`" ./cmd/go-semantic-release/
+
+    - name: Release
+      env:
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+      run: ./build/go-semantic-release-temp release --loglevel trace

internal/cache/cache_test.go

@@ -3,13 +3,14 @@ package cache_test
 import (
 	"testing"
 
+	"io/ioutil"
+	"os"
+	"path"
+
 	"github.com/Masterminds/semver"
 	"github.com/Nightapes/go-semantic-release/internal/cache"
 	"github.com/Nightapes/go-semantic-release/internal/shared"
 	"github.com/stretchr/testify/assert"
-	"io/ioutil"
-	"os"
-	"path"
 )
 
 func TestReadCacheNotFound(t *testing.T) {
@@ -54,7 +55,6 @@ func TestWriteAndReadCache(t *testing.T) {
 			VersionString: "1.1.0",
 		},
 		Branch: "master",
-		Draft:  true,
 		Commits: map[shared.Release][]shared.AnalyzedCommit{
 			"major": []shared.AnalyzedCommit{
 				shared.AnalyzedCommit{
@@ -97,7 +97,6 @@ func TestWriteNotFound(t *testing.T) {
 			Version: createVersion("1.1.0"),
 		},
 		Branch: "master",
-		Draft:  true,
 	})
 	assert.Errorf(t, err, "Write non exsiting file")
 

internal/calculator/calculator.go

@@ -40,29 +40,24 @@ func (c *Calculator) IncPrerelease(preReleaseType string, version *semver.Versio
 }
 
 //CalculateNewVersion from given commits and lastversion
-func (c *Calculator) CalculateNewVersion(commits map[shared.Release][]shared.AnalyzedCommit, lastVersion *semver.Version, releaseType string, firstRelease bool) (semver.Version, bool) {
+func (c *Calculator) CalculateNewVersion(commits map[shared.Release][]shared.AnalyzedCommit, lastVersion *semver.Version, releaseType string, firstRelease bool) semver.Version {
 	switch releaseType {
-	case "beta", "alpha":
+	case "beta", "alpha", "rc":
 		if len(commits["major"]) > 0 || len(commits["minor"]) > 0 || len(commits["patch"]) > 0 {
 			version, _ := c.IncPrerelease(releaseType, lastVersion)
-			return version, true
-		}
-	case "rc":
-		if len(commits["major"]) > 0 || len(commits["minor"]) > 0 || len(commits["patch"]) > 0 {
-			version, _ := c.IncPrerelease(releaseType, lastVersion)
-			return version, false
+			return version
 		}
 	case "release":
 		if !firstRelease {
 			if len(commits["major"]) > 0 {
-				return lastVersion.IncMajor(), false
+				return lastVersion.IncMajor()
 			} else if len(commits["minor"]) > 0 {
-				return lastVersion.IncMinor(), false
+				return lastVersion.IncMinor()
 			} else if len(commits["patch"]) > 0 {
-				return lastVersion.IncPatch(), false
+				return lastVersion.IncPatch()
 			}
 		}
 	}
 
-	return *lastVersion, false
+	return *lastVersion
 }

internal/calculator/calculator_test.go

@@ -72,7 +72,6 @@ func TestCalculator_CalculateNewVersion(t *testing.T) {
 		releaseType     string
 		lastVersion     *semver.Version
 		nextVersion     string
-		isDraft         bool
 		isFirst         bool
 		analyzedCommits map[shared.Release][]shared.AnalyzedCommit
 	}{
@@ -90,7 +89,6 @@ func TestCalculator_CalculateNewVersion(t *testing.T) {
 				"none":  []shared.AnalyzedCommit{},
 			},
 			isFirst: false,
-			isDraft: true,
 		},
 		{
 			testCase:    "version with preRelease beta",
@@ -106,7 +104,6 @@ func TestCalculator_CalculateNewVersion(t *testing.T) {
 				"none":  []shared.AnalyzedCommit{},
 			},
 			isFirst: false,
-			isDraft: true,
 		},
 		{
 			testCase:    "version without commits",
@@ -120,7 +117,6 @@ func TestCalculator_CalculateNewVersion(t *testing.T) {
 				"none":  []shared.AnalyzedCommit{},
 			},
 			isFirst: false,
-			isDraft: false,
 		},
 		{
 			testCase:    "version with commits and first release",
@@ -134,7 +130,6 @@ func TestCalculator_CalculateNewVersion(t *testing.T) {
 				"none":  []shared.AnalyzedCommit{},
 			},
 			isFirst: true,
-			isDraft: false,
 		},
 		{
 			testCase:    "version with commits and rc release",
@@ -148,7 +143,6 @@ func TestCalculator_CalculateNewVersion(t *testing.T) {
 				"none":  []shared.AnalyzedCommit{},
 			},
 			isFirst: false,
-			isDraft: false,
 		},
 		{
 			testCase:    "version with commits and rc release",
@@ -162,7 +156,6 @@ func TestCalculator_CalculateNewVersion(t *testing.T) {
 				"none":  []shared.AnalyzedCommit{},
 			},
 			isFirst: false,
-			isDraft: false,
 		},
 		{
 			testCase:    "version with commits and major release",
@@ -176,7 +169,6 @@ func TestCalculator_CalculateNewVersion(t *testing.T) {
 				"none":  []shared.AnalyzedCommit{},
 			},
 			isFirst: false,
-			isDraft: false,
 		},
 		{
 			testCase:    "version with commits and minor release",
@@ -190,7 +182,6 @@ func TestCalculator_CalculateNewVersion(t *testing.T) {
 				"none":  []shared.AnalyzedCommit{},
 			},
 			isFirst: false,
-			isDraft: false,
 		},
 		{
 			testCase:    "version with commits and minor patch",
@@ -204,15 +195,13 @@ func TestCalculator_CalculateNewVersion(t *testing.T) {
 				"none":  []shared.AnalyzedCommit{},
 			},
 			isFirst: false,
-			isDraft: false,
 		},
 	}
 
 	c := calculator.New()
 
 	for _, test := range testConfigs {
-		next, draft := c.CalculateNewVersion(test.analyzedCommits, test.lastVersion, test.releaseType, test.isFirst)
-		assert.Equalf(t, test.isDraft, draft, "Should have draft %t for testcase %s", test.isDraft, test.testCase)
+		next := c.CalculateNewVersion(test.analyzedCommits, test.lastVersion, test.releaseType, test.isFirst)
 		assert.Equalf(t, test.nextVersion, next.String(), "Should have version %s for testcase %s", test.nextVersion, test.testCase)
 	}
 

internal/ci/ci.go

@@ -42,6 +42,7 @@ func GetCIProvider(gitUtil *gitutil.GitUtil, envs map[string]string) (*ProviderC
 
 	services := []Service{
 		Travis{},
+		GithubActions{},
 		Git{gitUtil: gitUtil}, // GIt must be the last option to check
 	}
 

internal/ci/ci_test.go

@@ -89,6 +89,28 @@ func TestCi(t *testing.T) {
 			result:   &ci.ProviderConfig{IsPR: false, PR: "", PRBranch: "", Branch: "master", Tag: "TAG", Commit: "190bfd6aa60022afd0ef830342cfb07e33c45f37", BuildURL: "https://travis-ci.com/owner/repo/builds/1234", Service: "travis", Name: "Travis CI"},
 			hasError: false,
 		},
+		{
+			service: "Github Actions PR",
+			envs: map[string]string{
+				"GITHUB_EVENT_NAME": "pull_request",
+				"GITHUB_SHA":        "190bfd6aa60022afd0ef830342cfb07e33c45f37",
+				"GITHUB_REF":        "master",
+				"GITHUB_ACTION":     "action",
+			},
+			result:   &ci.ProviderConfig{IsPR: true, PR: "", PRBranch: "", Branch: "master", Tag: "", Commit: "190bfd6aa60022afd0ef830342cfb07e33c45f37", BuildURL: "", Service: "GithubActions", Name: "GithubActions CI"},
+			hasError: false,
+		},
+		{
+			service: "Github Actions Push",
+			envs: map[string]string{
+				"GITHUB_EVENT_NAME": "push",
+				"GITHUB_SHA":        "190bfd6aa60022afd0ef830342cfb07e33c45f37",
+				"GITHUB_REF":        "refs/heads/feature-branch-1",
+				"GITHUB_ACTION":     "action",
+			},
+			result:   &ci.ProviderConfig{IsPR: false, PR: "", PRBranch: "", Branch: "feature-branch-1", Tag: "", Commit: "190bfd6aa60022afd0ef830342cfb07e33c45f37", BuildURL: "", Service: "GithubActions", Name: "GithubActions CI"},
+			hasError: false,
+		},
 	}
 
 	for _, config := range testConfigs {

internal/ci/github_actions.go

@@ -0,0 +1,43 @@
+package ci
+
+import (
+	"fmt"
+	"strings"
+
+	log "github.com/sirupsen/logrus"
+)
+
+//GithubActions struct
+type GithubActions struct{}
+
+//Detect if on GithubActions
+func (t GithubActions) detect(envs map[string]string) (*ProviderConfig, error) {
+
+	if _, exists := envs["GITHUB_ACTION"]; !exists {
+		return nil, fmt.Errorf("not running on Github Actions")
+	}
+
+	isPR := false
+
+	value := envs["GITHUB_EVENT_NAME"]
+
+	if value == "pull_request" {
+		isPR = true
+	} else {
+		log.Debugf("GITHUB_EVENT_NAME=%s, not running on pr", value)
+	}
+
+	branch := envs["GITHUB_REF"]
+
+	if strings.HasPrefix(envs["GITHUB_REF"], "refs/heads/") {
+		branch = strings.Replace(branch, "refs/heads/", "", 1)
+	}
+
+	return &ProviderConfig{
+		Service: "GithubActions",
+		Name:    "GithubActions CI",
+		Commit:  envs["GITHUB_SHA"],
+		Branch:  branch,
+		IsPR:    isPR,
+	}, nil
+}

internal/releaser/github/github.go

@@ -32,7 +32,7 @@ type Client struct {
 func New(c *config.GitHubProvider) (*Client, error) {
 	var err error
 
-	if c.AccessToken, err = util.GetAccessToken(GITHUB); err != nil {
+	if c.AccessToken, err = util.GetAccessToken("GITHUB_TOKEN"); err != nil {
 		return &Client{}, err
 	}
 	ctx := context.Background()
@@ -96,7 +96,6 @@ func (g *Client) CreateRelease(releaseVersion *shared.ReleaseVersion, generatedC
 		TargetCommitish: &releaseVersion.Branch,
 		Name:            &generatedChangelog.Title,
 		Body:            &generatedChangelog.Content,
-		Draft:           &releaseVersion.Draft,
 		Prerelease:      &prerelease,
 	})
 	if err != nil {

internal/releaser/github/github_test.go

@@ -90,7 +90,6 @@ var testReleases = []testReleaseStruct{
 				Commit:  "bar",
 			},
 			Branch: "master",
-			Draft:  false,
 		},
 		generatedChangelog: &shared.GeneratedChangelog{
 			Title:   "title",
@@ -115,7 +114,6 @@ var testReleases = []testReleaseStruct{
 				Commit:  "bar",
 			},
 			Branch: "master",
-			Draft:  false,
 		},
 		generatedChangelog: &shared.GeneratedChangelog{
 			Title:   "title",
@@ -145,19 +143,19 @@ func initHTTPServer(respCode int, body string) *httptest.Server {
 func TestNew(t *testing.T) {
 	for _, testOject := range testNewClient {
 		if testOject.valid {
-			os.Setenv("GITHUB_ACCESS_TOKEN", "XXX")
+			os.Setenv("GITHUB_TOKEN", "XXX")
 		}
 
 		_, err := github.New(&testOject.config)
 		assert.Equal(t, testOject.valid, err == nil)
 
-		os.Unsetenv("GITHUB_ACCESS_TOKEN")
+		os.Unsetenv("GITHUB_TOKEN")
 
 	}
 }
 
 func TestGetCommitURL(t *testing.T) {
-	os.Setenv("GITHUB_ACCESS_TOKEN", "XX")
+	os.Setenv("GITHUB_TOKEN", "XX")
 	for _, testOject := range testNewClient {
 		client, _ := github.New(&testOject.config)
 		actualURL := client.GetCommitURL()
@@ -170,12 +168,12 @@ func TestGetCommitURL(t *testing.T) {
 			assert.EqualValues(t, expectedURL, actualURL)
 		}
 	}
-	os.Unsetenv("GITHUB_ACCESS_TOKEN")
+	os.Unsetenv("GITHUB_TOKEN")
 
 }
 
 func TestGetCompareURL(t *testing.T) {
-	os.Setenv("GITHUB_ACCESS_TOKEN", "XX")
+	os.Setenv("GITHUB_TOKEN", "XX")
 	for _, testOject := range testNewClient {
 		client, _ := github.New(&testOject.config)
 		actualURL := client.GetCompareURL("1", "2")
@@ -188,12 +186,12 @@ func TestGetCompareURL(t *testing.T) {
 			assert.EqualValues(t, expectedURL, actualURL)
 		}
 	}
-	os.Unsetenv("GITHUB_ACCESS_TOKEN")
+	os.Unsetenv("GITHUB_TOKEN")
 
 }
 
 func TestValidateConfig(t *testing.T) {
-	os.Setenv("GITHUB_ACCESS_TOKEN", "XX")
+	os.Setenv("GITHUB_TOKEN", "XX")
 	for _, testOject := range testHelperMethod {
 		client, _ := github.New(&testOject.config)
 		err := client.ValidateConfig()
@@ -201,11 +199,11 @@ func TestValidateConfig(t *testing.T) {
 		assert.Equal(t, testOject.valid, err == nil)
 
 	}
-	os.Unsetenv("GITHUB_ACCESS_TOKEN")
+	os.Unsetenv("GITHUB_TOKEN")
 }
 
 func TestCreateRelease(t *testing.T) {
-	os.Setenv("GITHUB_ACCESS_TOKEN", "XX")
+	os.Setenv("GITHUB_TOKEN", "XX")
 
 	for _, testObejct := range testReleases {
 		if testObejct.valid {
@@ -232,6 +230,6 @@ func TestCreateRelease(t *testing.T) {
 			assert.Error(t, err)
 		}
 	}
-	os.Unsetenv("GITHUB_ACCESS_TOKEN")
+	os.Unsetenv("GITHUB_TOKEN")
 
 }

internal/releaser/gitlab/gitlab_test.go

@@ -94,7 +94,6 @@ func TestCreateRelease(t *testing.T) {
 					Commit:  "bar",
 				},
 				Branch: "master",
-				Draft:  false,
 			},
 			generatedChangelog: &shared.GeneratedChangelog{
 				Title:   "title",
@@ -119,7 +118,6 @@ func TestCreateRelease(t *testing.T) {
 					Commit:  "bar",
 				},
 				Branch: "master",
-				Draft:  false,
 			},
 			generatedChangelog: &shared.GeneratedChangelog{
 				Title:   "title",
@@ -145,7 +143,6 @@ func TestCreateRelease(t *testing.T) {
 					Commit:  "bar",
 				},
 				Branch: "master",
-				Draft:  false,
 			},
 			generatedChangelog: &shared.GeneratedChangelog{
 				Title:   "title",

internal/releaser/releaser.go

@@ -2,6 +2,7 @@ package releaser
 
 import (
 	"fmt"
+	"strings"
 
 	"github.com/Nightapes/go-semantic-release/internal/releaser/github"
 	"github.com/Nightapes/go-semantic-release/internal/releaser/gitlab"
@@ -41,7 +42,7 @@ func (r *Releasers) GetReleaser() (Releaser, error) {
 		return github.New(&r.config.GitHubProvider)
 	case gitlab.GITLAB:
 		log.Debugf("initialize new %s-provider", gitlab.GITLAB)
-		accessToken, err := util.GetAccessToken(gitlab.GITLAB)
+		accessToken, err := util.GetAccessToken(fmt.Sprintf("%s_ACCESS_TOKEN", strings.ToUpper(gitlab.GITLAB)))
 		if err != nil {
 			return nil, err
 		}

internal/releaser/util/util.go

@@ -49,11 +49,9 @@ func NewAddHeaderTransport(T http.RoundTripper, key, value string) *AddHeaderTra
 }
 
 // GetAccessToken lookup for the providers accesstoken
-func GetAccessToken(providerName string) (string, error) {
+func GetAccessToken(envName string) (string, error) {
 	var token string
 	var exists bool
-	envName := fmt.Sprintf("%s_ACCESS_TOKEN", strings.ToUpper(providerName))
-
 	log.Debugf("check if %s environment variable is set", envName)
 
 	if token, exists = os.LookupEnv(envName); !exists {

internal/releaser/util/util_test.go

@@ -43,7 +43,7 @@ func TestGetAccessToken(t *testing.T) {
 			fmt.Println(err.Error())
 		}
 
-		_, err := util.GetAccessToken(testObject.providerName)
+		_, err := util.GetAccessToken(envName)
 
 		assert.Equal(t, testObject.valid, err == nil)
 		os.Unsetenv(envName)
@@ -193,7 +193,7 @@ func TestDoAndRoundTrip(t *testing.T) {
 			hasError:         true,
 			responseBody:     &example{},
 			responseBodyType: &example{},
-			path:             "broken",
+			path:             "4/broken",
 		},
 	}
 
@@ -214,6 +214,8 @@ func TestDoAndRoundTrip(t *testing.T) {
 
 		}))
 
+		defer testServer.Close()
+
 		req, err := http.NewRequest("POST", testServer.URL+testOject.path, nil)
 		assert.NoError(t, err)
 
@@ -225,6 +227,5 @@ func TestDoAndRoundTrip(t *testing.T) {
 			assert.Equal(t, testOject.statusCode, resp.StatusCode)
 			assert.Equal(t, testOject.responseBody, testOject.responseBodyType)
 		}
-		testServer.Close()
 	}
 }

internal/shared/shared.go

@@ -6,17 +6,16 @@ import (
 
 //ReleaseVersion struct
 type ReleaseVersion struct {
-	Last    ReleaseVersionEntry                            `yaml:"last"`
-	Next    ReleaseVersionEntry                            `yaml:"next"`
-	Branch  string                                         `yaml:"branch"`
-	Draft   bool                                           `yaml:"draft"`
+	Last    ReleaseVersionEntry          `yaml:"last"`
+	Next    ReleaseVersionEntry          `yaml:"next"`
+	Branch  string                       `yaml:"branch"`
 	Commits map[Release][]AnalyzedCommit `yaml:"commits"`
 }
 
 //ReleaseVersionEntry struct
 type ReleaseVersionEntry struct {
-	Commit        string `yaml:"commit"`
-	VersionString string `yaml:"version"`
+	Commit        string          `yaml:"commit"`
+	VersionString string          `yaml:"version"`
 	Version       *semver.Version `yaml:"-"`
 }
 

pkg/config/config.go

@@ -38,15 +38,15 @@ type GitLabProvider struct {
 
 // ReleaseConfig struct
 type ReleaseConfig struct {
-	CommitFormat          string            `yaml:"commitFormat"`
-	Branch                map[string]string `yaml:"branch"`
-	Changelog             ChangelogConfig   `yaml:"changelog,omitempty"`
-	Release               string            `yaml:"release,omitempty"`
-	GitHubProvider        GitHubProvider    `yaml:"github,omitempty"`
-	GitLabProvider        GitLabProvider    `yaml:"gitlab,omitempty"`
-	Assets                []Asset           `yaml:"assets"`
-	ReleaseTitle          string            `yaml:"title"`
-	IsPreRelease, IsDraft bool
+	CommitFormat   string            `yaml:"commitFormat"`
+	Branch         map[string]string `yaml:"branch"`
+	Changelog      ChangelogConfig   `yaml:"changelog,omitempty"`
+	Release        string            `yaml:"release,omitempty"`
+	GitHubProvider GitHubProvider    `yaml:"github,omitempty"`
+	GitLabProvider GitLabProvider    `yaml:"gitlab,omitempty"`
+	Assets         []Asset           `yaml:"assets"`
+	ReleaseTitle   string            `yaml:"title"`
+	IsPreRelease   bool
 }
 
 // Read ReleaseConfig

pkg/config/config_test.go

@@ -3,11 +3,12 @@ package config_test
 import (
 	"testing"
 
-	"github.com/Nightapes/go-semantic-release/pkg/config"
-	"github.com/stretchr/testify/assert"
 	"io/ioutil"
 	"os"
 	"path"
+
+	"github.com/Nightapes/go-semantic-release/pkg/config"
+	"github.com/stretchr/testify/assert"
 )
 
 func TestReadCacheNotFound(t *testing.T) {
@@ -88,7 +89,6 @@ github:
 				Compress: false}},
 		ReleaseTitle: "go-semantic-release release",
 		IsPreRelease: false,
-		IsDraft:      false,
 	}, result)
 
 }
@@ -105,7 +105,6 @@ github:
 // 			Version: createVersion("1.1.0"),
 // 		},
 // 		Branch: "master",
-// 		Draft:  true,
 // 	})
 // 	assert.Errorf(t, err, "Write non exsiting file")
 

pkg/semanticrelease/semantic-release.go

@@ -97,12 +97,11 @@ func (s *SemanticRelease) GetNextVersion(provider *ci.ProviderConfig, force bool
 
 	analyzedCommits := s.analyzer.Analyze(commits)
 
-	isDraft := false
 	var newVersion semver.Version
 	for branch, releaseType := range s.config.Branch {
 		if provider.Branch == branch || strings.HasPrefix(provider.Branch, branch) {
 			log.Debugf("Found branch config for branch %s with release type %s", provider.Branch, releaseType)
-			newVersion, isDraft = s.calculator.CalculateNewVersion(analyzedCommits, lastVersion, releaseType, firstRelease)
+			newVersion = s.calculator.CalculateNewVersion(analyzedCommits, lastVersion, releaseType, firstRelease)
 			break
 		}
 	}
@@ -117,7 +116,6 @@ func (s *SemanticRelease) GetNextVersion(provider *ci.ProviderConfig, force bool
 			Version: lastVersion,
 		},
 		Branch:  provider.Branch,
-		Draft:   isDraft,
 		Commits: analyzedCommits,
 	}
 
@@ -179,12 +177,12 @@ func (s *SemanticRelease) WriteChangeLog(changelogContent, file string) error {
 func (s *SemanticRelease) Release(provider *ci.ProviderConfig, force bool) error {
 
 	if provider.IsPR {
-		log.Debugf("Will not perform a new release. This is a pull request")
+		log.Infof("Will not perform a new release. This is a pull request")
 		return nil
 	}
 
 	if _, ok := s.config.Branch[provider.Branch]; !ok {
-		log.Debugf("Will not perform a new release. Current %s branch is not configured in release config", provider.Branch)
+		log.Infof("Will not perform a new release. Current %s branch is not configured in release config", provider.Branch)
 		return nil
 	}