-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Beta.6 Change log draft #4654
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
Beta.6 Change log draft #4654
Conversation
CHANGELOG.md
Outdated
* Fix Web CmdLets `-SkipHeaderValidation` to work with non-standard User-Agent headers. (#4479 & #4512) (Thanks @markekraus) | ||
* Use supported API to set Central Access Policy ID (CAPID) in SACL. (#4496) | ||
* Make `Start-Trace` support paths that require escaping in the underlying APIs (#3863) | ||
* Ignore case when binding PSReadline keyhandler functions (#4300) (Thanks @oising) |
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.
I think using the word 'casing' makes it more obvious what this is rather than misreading 'case' to mean 'instance'
CHANGELOG.md
Outdated
* Use supported API to set Central Access Policy ID (CAPID) in SACL. (#4496) | ||
* Make `Start-Trace` support paths that require escaping in the underlying APIs (#3863) | ||
* Ignore case when binding PSReadline keyhandler functions (#4300) (Thanks @oising) | ||
* Removing `#if CORECLR` enabled, `Enable-PSRemoting` and `Disable-PSRemoting` (#4538) |
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.
Is this the right PR #? #4538 marks some feature tests as pending not for the cmdlets you listed. Also, it seems that it's not worth mentioning per PR removal of #if CORECLR
statements.
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.
should be #2671 and I'll notice I missed some commits.
CHANGELOG.md
Outdated
* Ignore case when binding PSReadline keyhandler functions (#4300) (Thanks @oising) | ||
* Removing `#if CORECLR` enabled, `Enable-PSRemoting` and `Disable-PSRemoting` (#4538) | ||
* Enable WSManCredSSP cmdlets and add tests. (#4336) | ||
* Use .NET Cores implementation for shell execution. (#4523) |
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.
I think we should explicitly use the term 'ShellExecute' as 'shell execution' implies execution within powershell (as a shell)
CHANGELOG.md
Outdated
|
||
### Documentation | ||
|
||
* Added Ilya (@iSazonov) as a Maintainer. ($4365) |
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.
Typo with $
CHANGELOG.md
Outdated
@@ -2,45 +2,69 @@ | |||
|
|||
## v6.0.0-beta.6 - 2017-08-24 | |||
|
|||
### Breaking change | |||
|
|||
* Make invalid argument handling to -File and -Command consitent and make exit codes consistent with Unix standards (#4573) |
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.
Only the error message is consistent, the arguments are handled differently by design
CHANGELOG.md
Outdated
### Engine updates and fixes | ||
|
||
* Make resource loading to work with PowerShell SxS installation (#4139) | ||
* Add missing assemblies to TPA list to make Pwrshplughin.dll work (#4502) | ||
* Make sure running `powershell` starts instance of the current version of PowerShell. (#4481) | ||
* Make sure we only use Unicode output by default on Nano and IoT systems (#4074) | ||
* Enable `-WindowStyle` to work. (#4573) |
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.
Should say: powershell -windowstyle, otherwise it's not obvious. Also mention only on Windows
CHANGELOG.md
Outdated
* Fix SSH Remoting handling of KeyFileParameter when the path must be quoted. (#4529) | ||
* Make Web CmdLets use HTML meta charset attribute value, if present (#4338) | ||
* Move to .NET Core 2.0 (#4603) |
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.
You should say: .Net Core 2.0 final
CHANGELOG.md
Outdated
* tab completion. (#4560) | ||
* various types. (#4503) | ||
* CDXML CmdLets. (#4537) | ||
* Only allow packaging of powershell, if it was build from a repo at the root of the file system named powershell. (#4569 & #4600) |
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.
Build => built
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.
LGTM
No description provided.