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 a NERDTreeToggleVCS command to the vcs plugin #1060

Merged
merged 9 commits into from
Nov 22, 2019

Conversation

willfindlay
Copy link
Contributor

@willfindlay willfindlay commented Nov 22, 2019

Description of Changes

Added a new command NERDTreeToggleVCS to the vcs plugin that behaves exactly like NERDTreeToggle except attempt to root at VCS root when creating the tree for the first time.


New Version Info

Author's Instructions

  • Derive a new MAJOR.MINOR.PATCH version number. Increment the:
    • MAJOR version when you make incompatible API changes
    • MINOR version when you add functionality in a backwards-compatible manner
    • PATCH version when you make backwards-compatible bug fixes
  • Update CHANGELOG.md, following the established pattern.

Collaborator's Instructions

  • Review CHANGELOG.md, suggesting a different version number if necessary.
  • After merge, tag the merge commit, e.g. git tag -a 3.1.4 -m "v3.1.4" && git push origin --tags

nerdtree_plugin/vcs.vim Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@willfindlay
Copy link
Contributor Author

Thanks for the review @PhilRunninger. I've made your requested changes.

Copy link
Member

@PhilRunninger PhilRunninger left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks for following the PR template's instructions.

@PhilRunninger PhilRunninger merged commit 82b1649 into preservim:master Nov 22, 2019
@willfindlay willfindlay deleted the togglevcs branch November 22, 2019 16:31
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.

2 participants