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

VsCodeNeovim #1897

Closed
wants to merge 140 commits into from
Closed

VsCodeNeovim #1897

wants to merge 140 commits into from

Conversation

Chillee
Copy link
Member

@Chillee Chillee commented Jul 4, 2017

The neovim rewrite!

For design doc/decisions/limitations/a rough idea of what's done, check here:
https://slack-files.com/T0ENVGY1E-F62R31A5V-1be4436e6c

Pretty much everything is in neovim.ts and extension.ts

If anybody has suggestions on anything about the design doc, or they have questions about any design choices, I'd be happy to listen to them.

(While I'm working on this I'll likely be less responsive to issues. Many of the current bugs would be fixed by this pr so please understand that I have less motivation to fix them).

neovim.ts Outdated
return [row - 1, character - 1];
}

static async getCursorPos(): Promise<[number, number]> {
Copy link

Choose a reason for hiding this comment

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

Vim.nv.cursor will return the cursor position (there is no equivalent to getpos though

xconverge and others added 26 commits October 28, 2017 22:18
@jpoon
Copy link
Member

jpoon commented Jan 5, 2018

Going to close this PR as development of the neovim rewrite is over at: https://github.com/Chillee/VSCodeNeovim and therefore all discussion of neovim rewrite should move over there.

@Chillee do you want to move it under the VSCodeVim org? makes it more easily discoverable. You should have the rights to do that if you choose.

@jpoon jpoon closed this Jan 5, 2018
@karlhorky
Copy link
Contributor

@Chillee sounds awesome. Will any further updates about any future merge into VSCodeVim be posted to this issue? I'll subscribe!

@insidewhy
Copy link
Contributor

@karlhorky No work been done on it since a month before your comment.

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.