jparise/vim-graphql

Submit to vim/vim

ryu-ichiroh opened this issue ยท 4 comments

Hi jparise.

Do you want to try to add to vim/vim?

jparise/vim-graphql is the most popular plugin for GraphQL.
https://github.com/search?o=desc&q=vim+graphql&s=stars&type=Repositories

Now vim/vim is not support graphql, not even syntax highlighting.
https://github.com/vim/vim/tree/master/runtime/syntax

Of course the rights keep yours.

The following links are a case of rust-lang/rust.vim.

Thanks.

That's an interesting idea, @ryu-ichiroh, and not one that I had previously considered.

One potential downside: vim-graphql currently integrates with a few other projects for improved syntax highlighting, such as https://github.com/pangloss/vim-javascript. I think we'd need to give that up if the code was part of vim itself. It might not be a big loss overall, but it's a consideration.

We would also need to rewrite the current test suite in terms of vim's own test assertions rather than Vader. That could be a good project to do in general, although it's not a priority.

@jparise Thank you for your positive comment!

As for the integrations and testing, vim/vim would not need them and It's enough to import ftplugin/, idnent/, and syntax/.
(I understand that the integration is a great feature and testing make the project keep to reduce errors.)

I made a patch to make it easier for you to submit. https://github.com/ryu-ichiroh/vim/pull/1/files

Most of the code was taken over from vim-graphql, and modified the headers as brammool's comment.
vim/vim#1356 (comment)

Thanks, @ryu-ichiroh! I need to give this a little more thought to form a better understanding of what I'm actually signing up to maintain (๐Ÿ™‚), but it does seem like a good idea.