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

Add test for git + git lfs on Makefile #9753

Merged
merged 1 commit into from
Jan 14, 2020
Merged

Conversation

guillep2k
Copy link
Member

This PR adds a check for git and git lfs as a dependency to the more used tests (unit and integration).

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jan 13, 2020
@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #9753   +/-   ##
======================================
  Coverage    42.3%   42.3%           
======================================
  Files         598     598           
  Lines       78318   78318           
======================================
  Hits        33135   33135           
- Misses      41127   41128    +1     
+ Partials     4056    4055    -1
Impacted Files Coverage Δ
services/pull/temp_repo.go 31.62% <0%> (-2.57%) ⬇️
modules/queue/workerpool.go 41.2% <0%> (+1.28%) ⬆️

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 1751d5f...724c84b. Read the comment docs.

@GiteaBot GiteaBot 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 Jan 14, 2020
@zeripath zeripath merged commit 565eb18 into go-gitea:master Jan 14, 2020
@guillep2k guillep2k deleted the check-lfs branch January 14, 2020 00:08
@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
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants