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.

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

Ignore binary files #33

Closed
ohumbel opened this issue Jun 17, 2015 · 10 comments
Closed

Ignore binary files #33

ohumbel opened this issue Jun 17, 2015 · 10 comments
Assignees
@ohumbel
Labels
enhancement

Comments

@ohumbel
Copy link
Member

ohumbel commented Jun 17, 2015

To prevent repository bloat, migrators want to ignore binary files (if so configured).
The configuration should at least list some file types to ignore, such as .zip or .jar.

Some background ...

These are reasons why to avoid big repositories:
http://blogs.atlassian.com/2014/09/ci-git-repos/

These are tips how to handle big files (if not otherwise possible)
http://blogs.atlassian.com/2014/05/handle-big-repositories-git/
@ohumbel
Copy link
Member Author

ohumbel commented Jun 17, 2015

I started to implement a prototype

@ohumbel ohumbel self-assigned this Jun 17, 2015
@WtfJoke
Copy link
Member

WtfJoke commented Jun 17, 2015

Thanks! Sounds good 👍 🎈

@WtfJoke WtfJoke mentioned this issue Jun 18, 2015
Closed
@ohumbel
Copy link
Member Author

ohumbel commented Jul 3, 2015

Just pushed my work in progress as branch 'bigfiles' to my fork (https://github.com/ohumbel/rtc2git).
No live tests passed so far.

@ohumbel ohumbel removed their assignment Jul 9, 2015
@WtfJoke
Copy link
Member

WtfJoke commented Jul 10, 2015

I think its probably useful to replace the code https://github.com/rtcTo/rtc2git/blob/develop/gitFunctions.py#L65 with an extended version of splitoutputofgitstatusz (where modifier is part of the return value - I guess a simple object will do this job)

@ohumbel
Copy link
Member Author

ohumbel commented Aug 21, 2015

Live tests passed on branch 'bigfiles'.

@WtfJoke
Copy link
Member

WtfJoke commented Aug 24, 2015

Good. Ready for merge? 👍

@ohumbel
Copy link
Member Author

ohumbel commented Aug 24, 2015

Yes, soon ready to merge. I am just working on unit tests, and then ...

@ohumbel
Copy link
Member Author

ohumbel commented Aug 24, 2015

Unified parsing of the output of 'git status -z' is implemented with ohumbel@f9fcbc7

@ohumbel
Copy link
Member Author

ohumbel commented Aug 24, 2015

Pull request: #45

@WtfJoke
Copy link
Member

WtfJoke commented Aug 25, 2015

Pull request was merged with 5810698

@WtfJoke WtfJoke closed this as completed Aug 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ohumbel ohumbel

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ohumbel @WtfJoke