Skip to content

refactor(#2837): multi instance reload #2885

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Sep 1, 2024
Merged

Conversation

alex-courtis
Copy link
Member

fixes #2837

require("nvim-tree.explorer.watch").setup(opts)

Marks = require "nvim-tree.marks"
Clipboard = require "nvim-tree.actions.fs.clipboard"
end

M.Explorer = Explorer
Copy link
Member Author

Choose a reason for hiding this comment

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

Flattened into a single class.

end
end

---TODO #2837 #2871 move this and similar to node
Copy link
Member Author

Choose a reason for hiding this comment

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

We'll need explorer aware nodes.

@@ -42,27 +48,24 @@ function Explorer.new(path)
end

---@class Explorer
local explorer = setmetatable({
local o = setmetatable({
Copy link
Member Author

Choose a reason for hiding this comment

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

Follow pattern elsewhere.

@alex-courtis alex-courtis requested a review from raaymax August 25, 2024 05:51
@alex-courtis
Copy link
Member Author

Successful dogfooding for a week.

@alex-courtis alex-courtis merged commit ea55ef1 into master Sep 1, 2024
5 checks passed
@alex-courtis alex-courtis deleted the reload-multiinstance branch September 1, 2024 05:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multi Instance: Refactor: move reload into Explorer
1 participant