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

feat: adds option to pass a getProcessCommandLine function #125

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jlcarvalho
Copy link

@jlcarvalho jlcarvalho commented Jul 20, 2024

In some cases it may be necessary to use a custom function to retrieve League's command line string, since the current methods (Get-CimInstance and wmic) rely on WMI, which may be unavailable (or broken) on some Windows machines.

This PR implements @junlarsen's suggestion from the discussion in #124.

@jlcarvalho jlcarvalho force-pushed the master branch 3 times, most recently from 9412389 to 2ba1ee3 Compare July 22, 2024 03:35
@jlcarvalho jlcarvalho changed the title feat: adds option to retrieve LeagueClientUx.exe process without WMI feat: adds option to pass a custom getProcessCommandLine function Jul 22, 2024
@jlcarvalho jlcarvalho changed the title feat: adds option to pass a custom getProcessCommandLine function feat: adds option to pass a getProcessCommandLine function Jul 22, 2024
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.

1 participant