Skip to content

Conversation

@shioyama
Copy link
Contributor

Great plugin!

Just one thing, when I make github links I want them to be permanent links, and to do that we need the commit hash not the branch name. I've made a minor change to do that.

Branches go out of date when changes are made, commits do not.
Copy link
Owner

Choose a reason for hiding this comment

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

@shioyama good catch! Thank you. Could you please also change the variable name branch to commit?

I'll merge after that!

Copy link
Owner

Choose a reason for hiding this comment

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

@shioyama I'll merge now and take care of it since I already have a few other changes in queue :)

tonchis added a commit that referenced this pull request Feb 23, 2015
Use commit rather than branch in url.
@tonchis tonchis merged commit f58e935 into tonchis:master Feb 23, 2015
@tonchis
Copy link
Owner

tonchis commented Feb 23, 2015

@shioyama Thanks for the patch :) FYI: https://github.com/tonchis/vim-to-github/releases/tag/v1.0.1

@shioyama
Copy link
Contributor Author

Great thanks!

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