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

Summary: Fixes #764 - Revert "some useful mappings I use everyday" #787

Merged
merged 1 commit into from
Jul 16, 2015

Conversation

toejough
Copy link
Contributor

@toejough toejough commented Jun 3, 2015

Problem:
The "useful mappings" override existing, arguably more useful
mappings.

Analysis:
Revert the commit.
This commit reverts commit 96678bc.

Testing:
Used the old mappings successfully.

Documentation:
No change - the reverted commit did not add/modify docs.

@aVolpe
Copy link

aVolpe commented Jun 3, 2015

+1

@toejough
Copy link
Contributor Author

toejough commented Jun 8, 2015

poke. @spf13 - what do you think of this pull request?

@j1mr10rd4n
Copy link
Contributor

+1
the mappings added in 96678bc would seem to belong in a user's own .vimrc.local instead of setting them for all users here

@MattWoelk
Copy link
Contributor

+1

Problem:
The "useful mappings" override existing, arguably *more* useful
mappings.

Analysis:
Revert the commit.
This commit reverts commit 96678bc.

Testing:
Used the old mappings successfully.

Documentation:
No change - the reverted commit did not add/modify docs.
@toejough
Copy link
Contributor Author

anyone with merge permissions want to merge this? Or comment as to why you don't want to merge it? It's a pretty small update, to revert some keybindings that override other keybindings that occur earlier in the rc.

spf13 added a commit that referenced this pull request Jul 16, 2015
Summary: Fixes #764 - Revert "some useful mappings I use everyday"
@spf13 spf13 merged commit 4543080 into spf13:3.0 Jul 16, 2015
@spf13
Copy link
Owner

spf13 commented Jul 16, 2015

Apologize for the delay.. DockerCon/GopherCon/OSCON have been quite all encompassing of my time. Merged now.

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.

None yet

5 participants