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

Feature: Register-ArgumentCompleter #25

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

BinToss
Copy link

@BinToss BinToss commented Nov 24, 2021

Heavily influenced by npm-completion 0.1.0

TODO

  • remove TabExpansion replacers
  • refactor formerly exported params
  • use list of Scoop apps for argument completion, based on npm-completion
  • review functionality. Does it work as intended?

Related Issues

Resolved #22, #24

@Moeologist
Copy link
Owner

Thanks for your work. TabExpansion replacer is an awful trick but I know nothing about Register-ArgumentCompleter
. BTW, I noticed that EOL was changed to LF (Unix). It may cause some compatibility issues?

@brian6932
Copy link

@Cologler This works when running. $CompletionsDir\commands.ps1, (with the module lines commented out ofc) but it doesn't work with PSLazyCompletion on pwsh launch, do you know how to fix that?

@BinToss
Copy link
Author

BinToss commented May 11, 2024

If it isn't obvious, my priorities have changed and I don't know when I'll come back to this.

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.

support Register-ArgumentCompleter mode
3 participants