| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
Steve Lee (@SteveL-MSFT) Why do you start to add prefixes to PR titles? This is contrary to our conventions. |
Sorry, something went wrong.
…ject.Tests.ps1 Co-Authored-By: Ilya <darpa@yandex.ru>
|
Ilya (@iSazonov) trying something based on an article I read about good commit messages. I suppose the prefix can be just in the commit and not in the PR title. |
Sorry, something went wrong.
|
Steve Lee (@SteveL-MSFT) All that we can not formalize is a human factor. I mean, something that can make things clearer can also be misleading. In this case, the simpler the better. So better not to have prefixes at all. |
Sorry, something went wrong.
|
Steve Lee (@SteveL-MSFT) :bug: (🐛) -gt BUG: 😉 https://gitmoji.carloscuesta.me/ But regardless, I agree that for commit messages having an easily-identifiable prefix can be a nice-to-have. Whether that's emoji or text is, I suppose, up to you guys. 😄
|
Sorry, something went wrong.
|
🎉v7.0.0-preview.6 has been released which incorporates this pull request.:tada: Handy links: |
Sorry, something went wrong.
…lated in `-ErrorVariable` (PowerShell#10840)
| Back | FazBrowse Home | New Git URL |
PR Summary
StopUpstreamCommandsException is a special exception used by cmdlets to tell PowerShell to stop upstream commands as it doesn't need any additional input. For example: "1,2,3 | select-object -first 1" should stop the first part of the pipeline after select-object received 1 object as the request has been fulfilled. However, if you use 1,2,3 | select-object -first 1 -errorvariable err, the internal exception (which is handled) is put into the $err variable (although $error it not populated). This means that scripts may treat this as an error or special case this System error and throwing it away.
The fix is in the code where the engine populates -ErrorVariable to discard StopUpstreamCommandsException.
I don't believe this is a breaking change as anyone working around this by throwing away that specific error (because select-object actually succeeded) won't be impacted by this change.
PR Context
Fix #9185
PR Checklist