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

feat: command autocomplete #40

Closed
wants to merge 2 commits into from

Conversation

alvarosevilla95
Copy link
Contributor

Adds basic completion for the Magenta command. This only completes the command action, but doesn't include advanced stuff like file completion for Magenta contexxt-files

"paste-selection",
}

M.command_complete = function(_, line)
Copy link
Owner

Choose a reason for hiding this comment

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

I'm not immediately following the logic here, I think I'll need to read it a bit more closely later. I think some more clear naming could help... or at least some comments explaining the shape of the objects we're expecting, or why we're mutating the parts table the way that we are.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah the logic is more convoluted that I'd like ngl. I used lazy.nvim complete function as reference. I've added some comments to the code to explain what it's actually doing

dlants added a commit that referenced this pull request Feb 4, 2025
@dlants
Copy link
Owner

dlants commented Feb 4, 2025

superseded by 9c5d051

@dlants dlants closed this Feb 4, 2025
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