-
-
Notifications
You must be signed in to change notification settings - Fork 321
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
gitui
uses gitoxide
#1078
Comments
This comment was marked as resolved.
This comment was marked as resolved.
Fixed, thanks! |
@Byron I just found this issue, and I would like to start contributing to Sorry in advance if this is not the best conversation channel, in which case I’d ask you to please refer me to a better one. :-) |
Thanks so much for offering, I think this is a great place to talk! During The Merge @extrawurst and I talked about It would fulfil your criteria and could be placed into its very own plumbing crate, isolating it neatly while making you acquainted with the lower-level building blocks of this project. Lower-level doesn't mean it's difficult, it probably only means that it's simple, with fancy stuff and integrations coming in with higher-levels. This really helps with focus, too. It's also not an easy task, but I'd be there to help along the way once a draft PR is up. Here are the features I think it would have to fulfil:
From a Please note that all building blocks you'd need area already available, like:
That's my brain dump about the feature, and is what I'd do if I wanted to bring it into I'd also offer a 1h zoom session to get you started, and you can send me an email if that interest you. Fun fact: this is something I can do because I talked to @extrawurst on The Merge, it's so useful to meet in person from time to time :). |
@Byron thanks for the brain dump, thats exactly what I discussed with @cruessler (also in person the day after merge - can only agree how valuable this is)! We agreed that ideally we lay out the interface we would want for this by working backwards and making the current blame (using libgit2) in |
Required Features
git status
submodule.update()
.git/config
, needs high-level branch rename and writing git configurationhttps://github.com/extrawurst/gitui
The text was updated successfully, but these errors were encountered: