Skip to content

Add installation procedure for vim-plug#146

Merged
cxw42 merged 1 commit intoeditorconfig:masterfrom
Nek-:patch-1
Apr 7, 2020
Merged

Add installation procedure for vim-plug#146
cxw42 merged 1 commit intoeditorconfig:masterfrom
Nek-:patch-1

Conversation

@Nek-
Copy link
Copy Markdown
Contributor

@Nek- Nek- commented Apr 4, 2020

No description provided.

@cxw42
Copy link
Copy Markdown
Member

cxw42 commented Apr 5, 2020

Thanks for this contribution! I think the number of options has outgrown "OR" :) . Would you be willing to add subheaders for each item (manual, Vim8, pathogen, Vundle, vim-plug)? E.g.,

...
### Vundle
Use [Vundle] by ...

### vim-plug
Use [vim-plug] by ...
...

Then I'll be happy to merge. Much appreciated!

Comment thread README.md Outdated
@Nek-
Copy link
Copy Markdown
Contributor Author

Nek- commented Apr 6, 2020

@cxw42 fixed.

@cxw42
Copy link
Copy Markdown
Member

cxw42 commented Apr 6, 2020

Looks good! One comment.

@cxw42 cxw42 self-requested a review April 6, 2020 14:39
Copy link
Copy Markdown
Member

@cxw42 cxw42 left a comment

Choose a reason for hiding this comment

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

One typo, and then I'll merge it!

Comment thread README.md Outdated
It also adds a little refactoring asked inside the pull request.
@Nek-
Copy link
Copy Markdown
Contributor Author

Nek- commented Apr 7, 2020

@cxw42 👍

@cxw42 cxw42 merged commit 8aff78a into editorconfig:master Apr 7, 2020
@cxw42
Copy link
Copy Markdown
Member

cxw42 commented Apr 7, 2020

Thanks again!

@Nek- Nek- deleted the patch-1 branch April 7, 2020 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants