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 #391, #372, #394, and some more #392

Merged
merged 10 commits into from
Nov 18, 2024
Merged

Fix #391, #372, #394, and some more #392

merged 10 commits into from
Nov 18, 2024

Conversation

cameronr
Copy link
Collaborator

@cameronr cameronr commented Nov 11, 2024

Contains

cameronr and others added 10 commits November 11, 2024 10:57
The fallback case for SessionSearch when Telescope isn't installed
wasn't setting the argument to handle_autosession_command correctly
If cwd_change_handling is enabled and the session being restored
contains a terminal, we might get a "Invalid argument:
buftype=terminal". Rather than restoring the session in the callback, we
schedule the restore for the next run of the event loop.
Since cwd_change_handling now restores in vim.schedule, we have to wait
for the next event loop in the tests. `vim.wait` works but using the
async testing is probably the better way in the future:

nvim-lua/plenary.nvim#271
@cameronr cameronr changed the title fix: #391 err loading session w/ terminal from Fzf Fix #391, #372, #394 Nov 17, 2024
@cameronr cameronr changed the title Fix #391, #372, #394 Fix #391, #372, #394, and some more Nov 17, 2024
@rmagatti rmagatti self-requested a review November 18, 2024 20:38
Copy link
Owner

@rmagatti rmagatti left a comment

Choose a reason for hiding this comment

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

Nice!

@cameronr cameronr merged commit 8d2eddb into rmagatti:main Nov 18, 2024
7 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.

[FEATURE] Show notification on auto session restore
2 participants