RamblingCookieMonster / PSDepend

PowerShell Dependency Handler
MIT License
285 stars 76 forks source link

Non-Windows Platform Support for GitHub Script #72

Closed dargmuesli closed 6 years ago

dargmuesli commented 6 years ago

That's what I needed to change to make the GitHub script work with PowerShellCore (6) on Linux. Please review and comment.

dargmuesli commented 6 years ago

It looks like I need to make further changes as PowerShell Core uses folder named PowerShell instead of WindowsPowerShell. I just noticed that.

dargmuesli commented 6 years ago

I've taken care of Core edition's differences and generally optimized the script. I'll use it and see if anything else needs to be changed.

RamblingCookieMonster commented 6 years ago

Hiyo! Sorry, been a bit tied up! Starting to look through this now!

dargmuesli commented 6 years ago

No problem, same for me :)

dargmuesli commented 6 years ago

Everything works for me now. I'd say that this can be merged, but of course I'll take care of any further wanted improvements.

dargmuesli commented 6 years ago

I just noticed a flaw with versioned imports. Will take care of this soon.

dargmuesli commented 6 years ago

Fixed! And a hint: AppVeyor allows builds on Ubuntu as a beta feature. Just in case you want to try it, write them an email and test it in a branch.

RamblingCookieMonster commented 6 years ago

Alrighty, thanks for putting this in! Merging, going to do a few things before deploying, probably later tonight