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

feat(git): Change Commit Log to use Lazygit #2959

Merged
merged 3 commits into from
May 27, 2024

Conversation

georgeguimaraes
Copy link
Contributor

@georgeguimaraes georgeguimaraes commented Apr 8, 2024

I found that using Lazygit Commit Log is much more helpful. You can have all the Lazygit usefulness here.

I'm using it to browse the last commits, and then browse again the files that were changed in those commits, and using the e shortcut to open the file in the same Neovim instance.

@dpetka2001
Copy link
Contributor

Really nice change. Hope it gets merged!! 😄

@@ -158,7 +158,6 @@ return {
{ "<leader>fr", "<cmd>Telescope oldfiles<cr>", desc = "Recent" },
{ "<leader>fR", LazyVim.telescope("oldfiles", { cwd = vim.uv.cwd() }), desc = "Recent (cwd)" },
-- git
{ "<leader>gc", "<cmd>Telescope git_commits<CR>", desc = "Commits" },
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you'd like to not remove a mapping, <leader>gl is free and makes sense to me.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But they do the same thing. They show the commits. It doesn't make sense to have 2 different mappings for 2 plugins that do the same thing essentially (to me personally at least).

@folke folke force-pushed the gg-change-commit-log-to-lazygit branch from 7d8b4ea to 0f2f3bf Compare May 27, 2024 05:53
@folke
Copy link
Collaborator

folke commented May 27, 2024

I kept the original <leader>gc and added a <leader>gl for LazyGit Log

@folke folke merged commit 9ccdbf6 into LazyVim:main May 27, 2024
3 checks passed
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.

None yet

5 participants