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.