Skip to content
This repository has been 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.

None yet

4 participants