Skip to content

Conversation

@ten3roberts
Copy link
Member

This fixes the error when updating Neogit through an auto-indexing plugin manager such as lazy. See #628 (reply in thread)

For the users they may need to once more do the git restore doc/tags dance as docs/tags would be in the state of modified by us, deleted by them or UD

This fixes the error when updating Neogit through an auto-indexing
plugin manager such as lazy. See #628 (reply in thread)
@ten3roberts
Copy link
Member Author

:Lazy sync

image

Caused by going from master to the remove-tags branch in plugin spec and updating (after removing local dev overrides)

@ten3roberts
Copy link
Member Author

This will be a breaking change, so I added the requested tracking issue for those

@ten3roberts
Copy link
Member Author

@CKolkey what do you think about this breaking change?

auto-session had to do a similar breaking change a few months ago

@ten3roberts ten3roberts requested a review from CKolkey July 20, 2023 13:56
@CKolkey
Copy link
Member

CKolkey commented Jul 21, 2023

I'm totally fine with this, but I'm not sure how this is a breaking change. What will users need to do? I've been using auto-session and have no recollection of needing to do anything to fix the docs..

@ten3roberts
Copy link
Member Author

The breaking commit is that users may need to go and explicitely restore the repo inside ~/.local/share/nvim/lazy/neogit as lazy will fail to pull as tags may be changed locally due to the aforementioned auto-indexing

@ten3roberts ten3roberts merged commit ed2a66c into master Jul 27, 2023
@CKolkey CKolkey deleted the remove-tags branch July 27, 2023 20:17
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.

3 participants