Skip to content

Sleep induced slowness #159

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Sleep induced slowness #159

wants to merge 1 commit into from

Conversation

thibaultmeyer
Copy link
Contributor

This pull request remove usage of sleep function who slow down git repos browsing (Issue #18).

Moved from gogs/git-module#24

@thibaultmeyer thibaultmeyer added this to the 1.0.0 milestone Nov 12, 2016
@thibaultmeyer thibaultmeyer added type/bug issue/critical This issue should be fixed ASAP. If it is a PR, the PR should be merged ASAP labels Nov 12, 2016
@codecov-io
Copy link

codecov-io commented Nov 12, 2016 •
edited
Loading

Current coverage is 3.03% (diff: 100%)

Merging #159 into master will not change coverage

@@            master      #159   diff @@
========================================
  Files           33        33          
  Lines         8100      8100          
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           246       246          
  Misses        7834      7834          
  Partials        20        20

Powered by Codecov. Last update b339858...eb7213a

@strk
Copy link
Member

strk commented Nov 12, 2016

Please move to https://github.com/go-gitea/git

@strk strk closed this Nov 12, 2016
@thibaultmeyer
Copy link
Contributor Author

thibaultmeyer commented Nov 12, 2016 •
edited
Loading

already merged on go-gitea 2 days ago, the main question is why fix is not reported here ?

@thibaultmeyer thibaultmeyer reopened this Nov 12, 2016
@lunny
Copy link
Member

lunny commented Nov 12, 2016

so that once your pr in git ismerged you have to update vendor

@thibaultmeyer
Copy link
Contributor Author

@lunny how to do that ?

@tboerger
Copy link
Member

Use https://github.com/kardianos/govendor/blob/master/README.md to update the deps properly and open a new pr.

@tboerger tboerger closed this Nov 12, 2016
@tboerger tboerger removed this from the 1.0.0 milestone Nov 12, 2016
@tboerger tboerger added reviewed/invalid and removed type/bug issue/critical This issue should be fixed ASAP. If it is a PR, the PR should be merged ASAP labels Nov 12, 2016
@thibaultmeyer
Copy link
Contributor Author

ok @tboerger just wait for the last PR open on go-gitea/git and i will submit a valid PR to update vendor

@thibaultmeyer thibaultmeyer deleted the fix/git-slow-sleep branch November 12, 2016 10:41
ethantkoenig added a commit to ethantkoenig/gitea that referenced this pull request Apr 29, 2017
@ethantkoenig @richmahn
Add changes from go-gitea#150 to .less files (go-gitea#159)
0637fe6
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
@delvh delvh added issue/not-a-bug The reported issue is the intended behavior or the problem is not inside Gitea and removed reviewed/invalid labels Oct 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
issue/not-a-bug The reported issue is the intended behavior or the problem is not inside Gitea
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@thibaultmeyer @codecov-io @strk @lunny @tboerger @delvh