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

Fix missing import in powershell plugin #674

Merged
merged 3 commits into from
Aug 7, 2019
Merged

Fix missing import in powershell plugin #674

merged 3 commits into from
Aug 7, 2019

Conversation

instinct-vfx
Copy link
Contributor

A missing import slipped into the last MR. I do think that the powershell plugin needs some more work. Most notably i would like it to properly escape paths on alias() calls. But that is way less trivial than it sounds at first. I also plan adding a second plugin for poweshell6+. It will be separate because it actually has differing feature sets and it also has a different executable name (pwsh).

The only functional change is the added import on top. The rest is auto-formatting for pep8 in vscode.

@nerdvegas
Copy link
Contributor

Hey Thorsten, would you mind dropping some TODO comments in so that the existing problems are identified? Cheers!

@instinct-vfx
Copy link
Contributor Author

Not at all, will go ahead asap. The powershell 6+ stuff is more a new ticket rather than comments as it will be a separate file. I also forgot another thing: I can possibly replace the existing registry hacks to get the clean system env with something more elegant in powershell. But i have to research fist. Will add TODO notes and report here.

@instinct-vfx
Copy link
Contributor Author

Realized i had another mishap. Fixed and added TODOs for the two things i have in mind. Will create an issue for pwsh support.

The fix has been tested with a few powershell ready packages i have that make proper use of the tweaked alias() setup.

@nerdvegas nerdvegas merged commit fc2c489 into AcademySoftwareFoundation:master Aug 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants