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

Vim plugin reorg #94

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Vim plugin reorg #94

wants to merge 3 commits into from

Conversation

pcapel
Copy link

@pcapel pcapel commented Jun 5, 2020

Reorganize and document the current plugins for vim.
We remove the mxw/vim-jsx plugin as it is deprecated and broken according to the README. The replacement is the suggested plugin.
Additionally we just removed outright the coffeescript plugin.

To help with organizing we added foldmethod=marker for vim files so that the vimbundle loads with some organizing folds around logically grouped plugins. This helps with readability with the inline documentation of the plugins.

Any feedback on the documentation is welcome. The goal was to try and synthesize the most useful parts of the plugins so that anyone could read through and get a sense of the rationale behind each plugin being a part of our core workflow.

Plugin removed: coffee script support (it's 2020!)
Altered plugin: JSX language support
@pcapel pcapel requested a review from avogel3 June 5, 2020 20:36
Plug 'adamlowe/vim-slurper'

" Support tab completion
" TODO: is there a better tab completion plugin?
Copy link
Member

@avogel3 avogel3 Jun 11, 2020

Choose a reason for hiding this comment

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

I would remove this TODO, and create an issue if you feel strongly about supertab.

A quick google search for "supertab alternative" yielded me the following 2 libraries. From a cursory glance, I'm a fan of the latter as it supports Neovim and it has pretty active development.

.vimbundle Outdated
" a = 1
" back = 3
"
" Which is a pattern that disgusts me. So I will be removing this to an
Copy link
Member

Choose a reason for hiding this comment

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

HAHA 😂

Copy link
Author

Choose a reason for hiding this comment

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

Oh shoot, Lol. I forgot to moderate my snark. I had originally planned for this to just be my modified version of dotmatrix. 😨

.vimbundle Outdated Show resolved Hide resolved
.vimbundle Outdated
Plug 'nelstrom/vim-textobj-rubyblock'
Plug 'pangloss/vim-javascript'

" You should be using Rg
Copy link
Member

Choose a reason for hiding this comment

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

Could you expand on this a little bit?

.vimbundle Outdated
Plug 'jgdavey/vim-weefactor'

" This is a plugin to allow user defined text objects, which is really friggin
Copy link
Member

@avogel3 avogel3 Jun 11, 2020

Choose a reason for hiding this comment

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

, which is really friggin

I would delete this

.vimbundle Outdated
Plug 'tpope/vim-commentary'

" Syntax highlighting and editing commands for cucumber files
" should go with things that I don't normally use
Copy link
Member

@avogel3 avogel3 Jun 11, 2020

Choose a reason for hiding this comment

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

remove this line

Edit: just 117

@avogel3
Copy link
Member

avogel3 commented Jun 11, 2020

Hey Phil, thanks for taking the time to put this together. I definitely laughed at a few of your extremely witty comments 😂 I'm not sure if you are done with the PR yet, however I already started to leave some feedback.

In addition to what I've already mentioned, I would request that you remove comments that state an opinion about a plugin. Other than that, looks good 👍

@pcapel pcapel marked this pull request as draft November 20, 2020 20:20
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.

3 participants