Alternative branching for custom indexes instead default branch about krew

[ardaguclu]

I added a solution with respect to my proposal #833. Appending branch name with # will set the cloned repositories default branch to the given one. It worked without any problem in my tests.

[chriskim06]

it seems a little strange to me to handle this with different versions of the index. homebrew handles this by allowing versioned formulae in their main tap. i dont know off the top of my head if that is already possible with krew (i forget exactly what parts of the plugin spec are used to setup the binary installed in .krew/store and the symlink in .krew/bin). there are other things in addition to having separate plugins that would be needed though, like how to link a specific version.

i think your implementation makes sense for solving this by versioning the index (as long as that clone command properly sets refs/remotes/origin/{branch} so that the git reset --hard @{upstream} is referencing the correct remote), but im not totally convinced it should be handled there instead of supporting different plugin versions.

[ardaguclu]

it seems a little strange to me to handle this with different versions of the index. homebrew handles this by allowing versioned formulae in their main tap. i dont know off the top of my head if that is already possible with krew (i forget exactly what parts of the plugin spec are used to setup the binary installed in .krew/store and the symlink in .krew/bin). there are other things in addition to having separate plugins that would be needed though, like how to link a specific version.

i think your implementation makes sense for solving this by versioning the index (as long as that clone command properly sets refs/remotes/origin/{branch} so that the git reset --hard @{upstream} is referencing the correct remote), but im not totally convinced it should be handled there instead of supporting different plugin versions.

@chriskim06 thanks for your comments. I had a similar discussion on Slack with @ahmetb and he also had some concerns about supporting this in terms of maintainability and narrow audience.

I'm closing my PR due to this, I'll keep issue for a while in case someone else also need this;
/close

[ardaguclu]

I incorrectly closed issue :). Sorry for noise.

[ardaguclu]

/close