Skip to content
This repository was archived by the owner on Apr 12, 2019. It is now read-only.

fix git version below 2.7 #119

Merged
merged 5 commits into from
May 26, 2018
Merged

fix git version below 2.7 #119

merged 5 commits into from
May 26, 2018

Conversation

daviian
Copy link
Member

@daviian daviian commented May 24, 2018

Targets go-gitea/gitea#3458

Should fix commitsBefore for older git versions.

repo_commit.go Outdated
@@ -9,6 +9,7 @@ import (
"container/list"
"strconv"
"strings"
"github.com/mcuadros/go-version"
Copy link
Member

@jonasfranz jonasfranz May 25, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix import order.

@daviian
Copy link
Member Author

daviian commented May 25, 2018

@JonasFranzDEV Actually it is not in Gopkg.toml, but it is in the lock file and vendor folder. Should this be added!?

@@ -0,0 +1,33 @@
package git
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comment header

"testing"

"github.com/stretchr/testify/assert"
"path/filepath"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

import order

Copy link
Member

@lunny lunny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please check my last comments.

@lunny
Copy link
Member

lunny commented May 25, 2018

maybe test different git version via .drone.yml?

@daviian
Copy link
Member Author

daviian commented May 25, 2018

@lunny Shouldn't drone already run tests for latest git and git 1.7?

@lunny
Copy link
Member

lunny commented May 25, 2018

@daviian Yes, there are already two git versions. LGTM

@lunny lunny merged commit 31f4b8e into go-gitea:master May 26, 2018
@daviian daviian deleted the bugfix/fix-repo-init branch May 26, 2018 06:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants