-
-
Notifications
You must be signed in to change notification settings - Fork 170
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
WIP: Add Redirections #75
Conversation
@corbob do you still plan on doing these? 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good so far, would be good to have the Tee-Object ones if you're able, otherwise we can take care of those in a follow up at some point. 😄
2 Error Stream PowerShell 2.0 | ||
3 Warning Stream PowerShell 3.0 | ||
4 Verbose Stream PowerShell 3.0 | ||
5 Debug Stream PowerShell 3.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These lines need to be standardized in their indentation levels here, it looks like a few of them ended up a little funky. 😄
$FileContent = Get-Content -Path $FilePath | ||
'__' | Should -Be $FileContent | ||
|
||
& {Write-Error 'The gasoline is in the water.'} 2> $FilePath |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A brief comment indicating the reason this is phrased differently would be a good idea, to minimise confusion.
Appreciate your work on this, I added some extra comments and put it in #148 💖 |
Just getting a PR in before I hunker down and make a bunch of work for Issue #24 (ignore that the branch is titled Issue14... apparently I can't computer at 5 AM)