-
Notifications
You must be signed in to change notification settings - Fork 7
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
Loading time too long #6
Comments
Is it possible to give me access to your blog source? I'll find time to try and make lighter git repository provider for you. |
I already fork blogit and temp fix it. After checking your source code,
just add the logic test if already has created_at will not call git_cli.
But the code not tidy, so didn't submit PR. The total loading dropped from
3 hrs to 1min.
…On Tue, Mar 10, 2020, 2:05 AM meddle ***@***.***> wrote:
Is it possible to give me access to your blog source? I'll find time to
try and make lighter git repository provider for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#6?email_source=notifications&email_token=AOL2XV7ZVRTSKX3QGM6Y46DRGXKBPA5CNFSM4LDGMHMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOKEN6A#issuecomment-596920056>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOL2XV4MLUZ6AK5ZTEB4TGTRGXKBPANCNFSM4LDGMHMA>
.
|
I refactored the code a bit, may already ok for a PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is same as issue#7 on blogit-web.
I only have 1800 posts in blog, but the initial loading time took 3 hours.
I think one thing at least go wrong is even already has timestamps in md file still call Git log to get it again.
The text was updated successfully, but these errors were encountered: