-
Notifications
You must be signed in to change notification settings - Fork 6
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
To resolve the conflict #28
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, there's only some minor changes you may need take a look
@@ -1,11 +1,12 @@ | |||
local bookmark_scope = require("bookmarks.domain.bookmark") | |||
local M = require("bookmarks.domain.bookmark") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
redefined local M
. line 1 and line 11
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, after merge, I replace all the bookmarks list name to M
@@ -33,9 +33,16 @@ end | |||
function M.find_bookmark_by_location(self, location) | |||
-- TODO: self location path | |||
for _, b in ipairs(self.bookmarks) do | |||
local b_type = M.get_value_type(b) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like this commit will fail the sign and line highlight.
And also the original pick method is not working as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there any exception
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the difference between tree.api
and tree.operate
?
Are they serve for different purposes?
edit:
Ok, my understanding is
operate
is treeview buffer local,
api
exposed to the outside as one plugin api
this commit has merge the bookmark-tree to main