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

Add basic remote module support #107

Merged
merged 1 commit into from
Jun 10, 2024
Merged

Add basic remote module support #107

merged 1 commit into from
Jun 10, 2024

Conversation

alexgenco
Copy link
Collaborator

@alexgenco alexgenco commented May 30, 2024

See: neovim/neovim#27949

This includes a helper method for defining remote modules as well as an acceptance spec to demonstrate their usage.

I chose to implement a new DSL class just for remote modules because the existing plugin DSL is far too complicated for simple RPC handling. As remote plugins are phased out, I expect to phase out and eventually deprecate the existing plugin DSL.

@alexgenco alexgenco marked this pull request as ready for review June 8, 2024 21:48
See: neovim/neovim#27949

This includes a helper method for defining remote modules as well as an
acceptance spec to demonstrate their usage.

I chose to implement a new DSL class just for remote modules because the
existing plugin DSL is far too complicated for simple RPC handling. As
remote plugins are phased out, I expect to phase out and eventually
deprecate the existing plugin DSL.
@alexgenco alexgenco merged commit 7e7b687 into main Jun 10, 2024
5 checks passed
@alexgenco alexgenco deleted the remote-module branch June 10, 2024 01:01
@justinmk justinmk mentioned this pull request Jun 10, 2024
6 tasks
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.

1 participant