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.

Sign up for GitHub

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

Jump to bottom

push optimizations (take 2) #1128

Merged
merged 7 commits into from
Apr 8, 2016
Merged

push optimizations (take 2) #1128

merged 7 commits into from
Apr 8, 2016

Commits on Apr 6, 2016

  1. teach git package how to get the local refs

    @technoweenie
    technoweenie committed Apr 6, 2016
    Configuration menu
    Copy the full SHA
    19c0e6c View commit details
    Browse the repository at this point in the history

  2. add ResolveRefs() placeholder for future optimization

    @technoweenie
    technoweenie committed Apr 6, 2016
    Configuration menu
    Copy the full SHA
    31dc0c2 View commit details
    Browse the repository at this point in the history

  3. introduce an uploadContext struct to share in push and pre-push

    @technoweenie
    technoweenie committed Apr 6, 2016
    Configuration menu
    Copy the full SHA
    abd9b1e View commit details
    Browse the repository at this point in the history

Commits on Apr 7, 2016

  1. no need for defer since there's only 1 return after cmd.Start()

    @technoweenie
    technoweenie committed Apr 7, 2016
    Configuration menu
    Copy the full SHA
    0935817 View commit details
    Browse the repository at this point in the history

  2. refactor *uploadContext, separate lib responsibilities vs cli respons… 

    …ibilities
    @technoweenie
    technoweenie committed Apr 7, 2016
    Configuration menu
    Copy the full SHA
    231686e View commit details
    Browse the repository at this point in the history

  3. these assertions should be more specific

    @technoweenie
    technoweenie committed Apr 7, 2016
    Configuration menu
    Copy the full SHA
    160ad66 View commit details
    Browse the repository at this point in the history

Commits on Apr 8, 2016

  1. rename AddUpload() to SetUploaded() for clarity

    @technoweenie
    technoweenie committed Apr 8, 2016
    Configuration menu
    Copy the full SHA
    be83f64 View commit details
    Browse the repository at this point in the history