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

added ArgumentCompleter for lyrics_stat param #430

Merged
merged 2 commits into from
Jun 5, 2023

Conversation

sharunkumar
Copy link
Contributor

This is an alternative to #428, without the issue in #429

I have tested it by invoking Install_Old_theme.bat in my test branch after replacing the urls to make sure the powershell script is running

@amd64fox
Copy link
Member

amd64fox commented Jun 4, 2023

Yes, it does seem like a problem with Invoke-Expression

Tell me how to make it work like shown in your gif from #428 , do I need to install an additional module or use a special terminal?

@sharunkumar
Copy link
Contributor Author

install an additional module or use a special terminal

it should work out of the box on Powershell, after typing .\Install.ps1 -lyrics_stat then press tab to auto complete

@sharunkumar
Copy link
Contributor Author

Oh also if you were asking about the prompt it's oh my posh: https://ohmyposh.dev/

@amd64fox amd64fox merged commit 4c35ef1 into SpotX-Official:main Jun 5, 2023
@amd64fox
Copy link
Member

amd64fox commented Jun 5, 2023

Oh damn, I didn't know Tab was used for that, thanks.

@sharunkumar sharunkumar deleted the ArgumentCompleter branch June 5, 2023 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants