/VisualStudio

[WIP] Generate Copy/Open link based on the latest commit #510

Open
wants to merge 2 commits into
from
+17 −1

Labels

WIP

Milestone

No milestone

Assignees

No one assigned

2 participants

@paladique @shana
@paladique
GitHub member
paladique commented Aug 19, 2016

Fixes #343

paladique added some commits Aug 19, 2016
@paladique paladique added method to get the latest pushed sha

Some checks were not successful

3 failing and 1 successful checks
Details
@hubot VisualStudio — Build #3897181 failed in 81s
Details
continuous-integration/appveyor/branch — AppVeyor build failed
Details
continuous-integration/appveyor/pr — AppVeyor build failed
Details
@github-cla GitHub CLA — @paladique has accepted the GitHub Contributor License Agreement.
62feac9
@paladique paladique spacing cleanup

Some checks were not successful

3 failing and 1 successful checks
Details
@hubot VisualStudio — Build #3897751 failed in 80s
Details
continuous-integration/appveyor/branch — AppVeyor build failed
Details
continuous-integration/appveyor/pr — AppVeyor build failed
Details
@github-cla GitHub CLA — @paladique has accepted the GitHub Contributor License Agreement.
9acae4f
@paladique paladique added the WIP label Aug 19, 2016
@paladique paladique changed the title from Generate Copy/Open link based on the latest commit to [WIP] Generate Copy/Open link based on the latest commit Aug 19, 2016
@shana
shana commented Aug 19, 2016

Probably worth checking whether the branch is tracking a remote branch and whether the commit hashes match there, and then maybe checking if any commits are reachable on that tracking branch, as an optimization, before going on enumerating all the remotes branches and all the local commits. Very likely the branch has been pushed at least once so that might hit a large chunk of use cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment