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

Use --allow-empty for performance, resolves #32 #42

Merged
merged 2 commits into from
Jun 28, 2016
Merged

Use --allow-empty for performance, resolves #32 #42

merged 2 commits into from
Jun 28, 2016

Conversation

numberoverzero
Copy link
Contributor

No description provided.

* Improves the size of the generated file:

2365211 new-gitfiti.sh
2529001 original-gitfiti.sh

* Cutting the number of executed commands in half (no more echo)
* Vastly reduces the amount of gc git performs during push
@numberoverzero
Copy link
Contributor Author

Sorry about the trailing whitespace changes, editor config.

@homeworkprod
Copy link
Collaborator

Seems good to me. I'd like to see this merged.

@numberoverzero
Copy link
Contributor Author

numberoverzero commented Jun 28, 2016

ping - anything I can do to help this along?

@gelstudios gelstudios merged commit 92c44e6 into gelstudios:master Jun 28, 2016
@gelstudios
Copy link
Owner

Nope I'm just getting back to non-work stuff, thanks for the PR 👍 merged.

@numberoverzero
Copy link
Contributor Author

thanks for merging! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants