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.

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

Add switch and restore completion to v0 #737

Closed
wants to merge 4 commits into from

Conversation

theaquamarine
Copy link
Collaborator

@theaquamarine theaquamarine commented Feb 8, 2020

As discussed in #733 (comment), this would allow v0 to support "new" usage for git at least until v1 updates/releases.

Uses the code from #702 for restore and switch support, plus a couple of lines from Get-GitDirectoryTests.ps1 to fix some tests being broken by git asking for user details.

Copy link
Owner

@dahlbyk dahlbyk left a comment

Choose a reason for hiding this comment

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

It's not immediately obvious to me why the remaining tests are failing. 🤔

$paramValues = $gitParamValues[$cmd][$param]

$completions = if ($paramValues -is [scriptblock]) {
& $paramValues $filter | Where-Object { $_ -like "$filter*" }
Copy link
Owner

Choose a reason for hiding this comment

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

Let's bring in fixes from #743

@rkeithhill
Copy link
Collaborator

@theaquamarine Given that beta4 is out now with support for these two new git commands, can we close this? Or if you'd like to get this merged, could you apply the fixes from #743 and see if that gets the tests to pass? Thanks.

@rkeithhill
Copy link
Collaborator

I'd like to close this now that v1 has shipped. Any objections?

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

Successfully merging this pull request may close these issues.

3 participants