search

sindrets / diffview.nvim

Single tabpage interface for easily cycling through diffs for all modified files for any git rev.
Other
3.93k stars 107 forks source link

feat: support `--walk-reflogs` for DiffviewFileHistory #396

Closed garcia5 closed 1 year ago

garcia5 commented 1 year ago

feat: support 'git log -g'

add support for -g, --walk-reflogs flag for the :DiffviewFileHistory command

Fixes https://github.com/sindrets/diffview.nvim/issues/353

$ git stash list
stash@{0}: WIP on feat/walk-reflogs: 334008a chore: add documentation for new options
stash@{1}: WIP on feat/walk-reflogs: 334008a chore: add documentation for new options

Before: image

After: image

sindrets commented 1 year ago

Thank you for the PR!

Really sorry about the late reply here. I've had little spare time the last couple weeks, and when I've taken time to work on the plugin, I've either been focusing on bugs, or working on some needed refactors.

Unfortunately I already have a branch where I've implemented this, alongside additional - in my opinion - needed features like actually showing reflog selectors in the panel (like stash@{0} etc...). I'm also not sure what would be the best way to actually show diffs for stashes, as they are actually merge commits. So I'm also experimenting with showing merge commits as 3-way diffs in the history view.

Sorry about not assigning myself to the issue. I hope you didn't spend too much time on this. I really appreciate that you're taking an interest in the project.

garcia5 commented 1 year ago

Thank you for the PR!

Really sorry about the late reply here. I've had little spare time the last couple weeks, and when I've taken time to work on the plugin, I've either been focusing on bugs, or working on some needed refactors.

Unfortunately I already have a branch where I've implemented this, alongside additional - in my opinion - needed features like actually showing reflog selectors in the panel (like stash@{0} etc...). I'm also not sure what would be the best way to actually show diffs for stashes, as they are actually merge commits. So I'm also experimenting with showing merge commits as 3-way diffs in the history view.

Sorry about not assigning myself to the issue. I hope you didn't spend too much time on this. I really appreciate that you're taking an interest in the project.

No problem! I have been using your plugin a lot, it's exactly what I've been looking for! Just wanted to help out if I could.

sindrets commented 1 year ago

Closing in favor of 413. Thank you for you work regardless!