-
Notifications
You must be signed in to change notification settings - Fork 483
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 git-crypt merge driver to support secret files merging (refactored #107) #180
base: master
Are you sure you want to change the base?
Conversation
Will this fix |
Any news here? I would love to see this merged but I also have to test it myself. |
Couple of years since my last comment on this feature.. Any plan to merge @AGWA? :) For anyone lurking, @jirutka's solution works like a charm - simply compile it, place it in your PATH and |
@AGWA This would be incredibly useful, as it addresses a common hinderance to using git-crypt. Is there any chance this could get merged? :) |
Maybe someone else can take over maintaining the project, There's 21 pull requests open, and this one is basically an essential feature. |
if anyone is interested, please review this PR, and you should be able to download the build from actions https://github.com/maxisam/git-crypt/actions/runs/11791593337 |
* feat: add merge support for git-crypt and update documentation AGWA#180 * fix: update download-artifact action to version 4 in Linux release workflow * chore: update github-script action to version 7 in release workflows
This PR is a refactoring of PR #107. The original PR mixes multiple unrelated changes in a single commit, including some maybe unwanted changes and code-style violations, which is probably the reason why it’s still not merged.
This PR contains only and only the changes needed for the merge driver support with minimum changes to the existing code (separated into an extra commit), so it should be easy for review. I’ve also fixed a few shortcomings in the original PR.