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

Improve tab expansion to handle params for push. #379

Merged
merged 5 commits into from
Feb 2, 2017
Merged

Improve tab expansion to handle params for push. #379

merged 5 commits into from
Feb 2, 2017

Commits on Feb 2, 2017

  1. Improve tab expansion to handle params for push. 

    Fix #234  BTW if we decide this is an OK fix (need another set of eyes on my $gitParams regex), I'd like to review the fact that all these switch regex case statements fallthrough to the next (observed while debugging tab expansion).  I don't believe that is necessary.  I'm thinking that most (all) of these should have a break statement.

Started a minimum set of Pester tests for tab completion.  Would be good to flesh these out over time.

Add debug configuration for VSCode that is dedicated to debugging Pester tests.
    @rkeithhill
    rkeithhill committed Feb 2, 2017
    Configuration menu
    Copy the full SHA
    080de0e View commit details
    Browse the repository at this point in the history

  2. Add more tab exp tests. Fix bug found by tests.

    @rkeithhill
    rkeithhill committed Feb 2, 2017
    Configuration menu
    Copy the full SHA
    61d51d4 View commit details
    Browse the repository at this point in the history

  3. Simplify push source:destination test inputs

    @dahlbyk @rkeithhill
    dahlbyk authored and rkeithhill committed Feb 2, 2017
    Configuration menu
    Copy the full SHA
    fc07ab6 View commit details
    Browse the repository at this point in the history

  4. Allow multiple refspecs in tab expansion. 

    Add Pester tests for this feature.
    @rkeithhill
    rkeithhill committed Feb 2, 2017
    Configuration menu
    Copy the full SHA
    e7b9312 View commit details
    Browse the repository at this point in the history

  5. Fix remote expansion test for multiple remotes 

    Make multiple push spec test more complete
    @dahlbyk @rkeithhill
    dahlbyk authored and rkeithhill committed Feb 2, 2017
    Configuration menu
    Copy the full SHA
    dffe385 View commit details
    Browse the repository at this point in the history