Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update git vendor #4059

Merged
merged 1 commit into from
May 27, 2018
Merged

update git vendor #4059

merged 1 commit into from
May 27, 2018

Conversation

daviian
Copy link
Member

@daviian daviian commented May 27, 2018

Updates git dependency to fix #3458

@bkcsoft bkcsoft added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label May 27, 2018
@codecov-io
Copy link

Codecov Report

Merging #4059 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4059   +/-   ##
=======================================
  Coverage   19.96%   19.96%           
=======================================
  Files         153      153           
  Lines       30478    30478           
=======================================
  Hits         6086     6086           
  Misses      23479    23479           
  Partials      913      913

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b660450...54716e5. Read the comment docs.

@lafriks lafriks added this to the 1.5.0 milestone May 27, 2018
@bkcsoft bkcsoft added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels May 27, 2018
@lafriks lafriks merged commit 0be2b34 into go-gitea:master May 27, 2018
@daviian daviian deleted the bugfix/home-repo-init branch May 27, 2018 19:00
@lunny
Copy link
Member

lunny commented May 29, 2018

maybe this should be back ported to release/v1.4

@daviian
Copy link
Member Author

daviian commented May 29, 2018

@lunny There are many other changes in the git repository that are not part of 1.4 yet. Specifically since commit go-gitea/git@827f97a
Don't know if this leads to problems?

@lunny
Copy link
Member

lunny commented May 29, 2018

@daviian I know and I don't know either.

daviian added a commit to daviian/gitea that referenced this pull request May 29, 2018
@lafriks lafriks added the backport/done All backports for this PR have been created label Jun 4, 2018
lafriks pushed a commit that referenced this pull request Jun 4, 2018
* update git vendor (#4059)

* fix errors resulting from git vendor update

* fmt
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Respository's home page not updated after first push
6 participants