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

fix: don't save gitrebase session #44

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

FineFindus
Copy link
Contributor

@FineFindus FineFindus commented Jan 13, 2024

Skip saving the session if it contains a gitrebase buffer. Similar to 8f7cbcc.

Steps to reproduce:

  1. Set nvim as $EDITOR.
  2. run git rebase --interactive and close nvim
  3. re-open nvim and notice how the gitrebase buffer is opened

@folke folke merged commit 9dbe264 into folke:main Jan 19, 2024
@folke
Copy link
Owner

folke commented Jan 19, 2024

thansk

@FineFindus FineFindus deleted the fix/gitrebase-session branch January 19, 2024 16:21
folke pushed a commit that referenced this pull request Jul 6, 2024
🤖 I have created a release *beep* *boop*
---


##
[3.0.0](v2.0.0...v3.0.0)
(2024-07-06)


### ⚠ BREAKING CHANGES

* opts.need specifieds how many buffers should be open for saving.
Replaces save_empty. Closes #19
* removed load_pre/load_post/save_pre/save_post in favor of events.
Check the readme
* removed `opts.options`. Use `vim.o.sessionoptions` instead.

### Features

* added `require("persistence").select()` to select a session to load
([5346b53](5346b53))
* opts.need specifieds how many buffers should be open for saving.
Replaces save_empty. Closes
[#19](#19)
([7bb5755](7bb5755))
* persistence.active. Check if a session saving is active
([f0ac0e9](f0ac0e9))
* **persistence:** add `pre-` and `post-` load hooks
([#24](#24))
([3d443bd](3d443bd))
* removed `opts.options`. Use `vim.o.sessionoptions` instead.
([eb5622e](eb5622e))
* removed load_pre/load_post/save_pre/save_post in favor of events.
Check the readme
([f58a838](f58a838))
* sessions per branch. Closes
[#9](#9)
([cd0054e](cd0054e))


### Bug Fixes

* don't save `gitrebase` session
([#44](#44))
([9dbe264](9dbe264))
* opts.need
([9c0e522](9c0e522))
* remove expand() as stdpath() expands itself
([#50](#50))
([a2fd3d9](a2fd3d9))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants