Skip to content

Merging to live to fix missing images #2047

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

Merged
merged 200 commits into from
Jan 12, 2018
Merged

Merging to live to fix missing images #2047

merged 200 commits into from
Jan 12, 2018

Conversation

sdwheeler
Copy link
Contributor

No description provided.

matt9ucci and others added 30 commits November 7, 2017 09:42
The example is a `-DisplayError` parameter example. But it uses `-ShowError` parameter.
The example is a -DisplayError parameter example. But it uses -ShowError parameter.
1. Replaced variable names with PascalCase convention and for consistency
2. Removed directory information [c:\] in the examples
3. Put powershell keyword for formatting examples
4. Replaced Invoke-RestMethod with Invoke-WebRequest in the Body example, added method Post
5. Removed InformationAction and InformationVariable parameters, which were mistakenly a copy of Body, included already in CommonVariables part
- Adds the website name to common node property inside configuration data.
- Also, it is now referenced (e.g. $Node.WebSiteName) in the DSC configuration document in MyWebApp.ps1.
Two set of changes made.
- Example scripts e.g. MyDscConfiguration.ps1 define the configuration inside them and call the configuration at the end line. Removed the call to the configuration at the end. Since the article demonstrates dot-sourcing the file and calling the configuration explicitly.
- Fixed typo in the file name (TEST-PC1.mof  to localhost.mof ), since the ComputerName was not specified it should default to using 'localhost' as the default argument for it.

P.S. - There is a comment on the web-page regarding the fixes made
updating Out-* commands with -NoNewLine per Issue 1739
* Updating the help content for v6.

* Address code review comments

* Fix broken links
Example number '7' is duplicated.
1. Removed statement that an alias is to an executable, the definition cotnains the path. It may - but does not need to be. For example:
Try Set-Alias np Notepad then look at the definition.
2. Clarified that an alias can also be to a powershell script (ps1 file).
3. Tidied up the language removing multiple 'And,' clauses to improve readability.
…roduced by PR #5043 (#1858)

* add documentation of new -AsHashTable switch for ConvertFrom-Json and also document the behaviour in case of duplicate strings.

* correct casing of -AsHashtable switch.

* Address PR comments about -AsHashtable switch for ConvertFrom-Json.md

* Accept pipeline input: False for -AsHashtable switch in ConvertFrom-Json.md

* Revert "Accept pipeline input: False for -AsHashtable switch in ConvertFrom-Json.md"
Accidentally change the wrong field.
This reverts commit 6e76191.

* Accept pipeline input: False for -AsHashtable switch in ConvertFrom-Json.md
Example number '4' is duplicated.
Sean Wheeler and others added 27 commits January 4, 2018 16:26
* Corrected Example 3

Example 3 is an example to remove the user from the group but instead it was removing the group completely.  This change adds the correct syntax to perform what was intended.

See #2001

* minor edits for spacing and metadata
* Enable running pandoc on about topic files

* Remove extra line

* Removed --ascii parameter from pandoc as it is not required
* Cmdlet help for ConvertFrom-Clixml and ConvertTo-Clixml for Powershell#3898

* minor edits

- Removed -information* common parameters
- reformat code blocks
- renumbered example

* minor updates

- reformatted examples
- removed common paramters

* update link to about topic

* minor edits

- removed common parameters
- formatted code blocks

* minor edits

- reformatted code blocks
- removed common parameters
* adding notes about OPS extensions

* added notes about OPS extensions and linking

* fix typo

* fix typo
* first pass of support lifecycle doc

* add notes on MIT license

* address slee's comments on support lifecycle

* respond to lifecycle feedback from Sean

* add lifecycle image

* Fixed spelling typo

* add lifecycle to TOC

* respond to feedback on lifecycle

* Removed en-us from URLs

You need to remove locale paths from URLs. Including en-us forces the reader to English rather than being redirected to the localized version of the article that matches their locale.

* add note on 12 months, fix en-us

* Fixed typo
This creates a couple new topics around WSMan remoting, SSH remoting, and installing/configuring PowerShell Core. It also adds a topic on using VS Code with PowerShell.
What's New in PowerShell Core 6.0
* update TOC and change title of setup mac/linux article

* fix typos
* attempt to fix images

* update to docfx to support images
* attempt to fix images

* update to docfx to support images

* update root docfx for images

* update docfx for image support
The original paged rendered badly - this edit is a move of a 'to do' comment in the MD so as to not break the sentence.
$PSVersionTable.OS is:
x [System.Environment]::OSVersion.VersionString
o [System.Runtime.InteropServices.RuntimeInformation]::OSDescription
The current example doesn't work for me when specifying the full file path.  It just outputs the file location rather than running the script.  I had to move to the location of the file and run it from there.

The parameter PowerShellVersion also doesn't appear to exist anymore so I have removed that.  

I've also changes file paths to match the 6.0.0 GA release rather than the previous alpha 9.
Updated Get-FormatData Markdown.  Fixes #1895
Updated Get-FormatData Markdown.  Fixes #1895
Updated Get-Runspace Markdown.  Fixes #1883
* attempt to fix images

* update to docfx to support images

* update root docfx for images

* update docfx for image support

* update docfx for images
@saldana
Copy link
Contributor

saldana commented Jan 12, 2018

✅ Validation status: passed

File Status Preview URL Details
dsc/docfx.json ✅Succeeded
reference/5.1/Microsoft.PowerShell.Utility/Get-FormatData.md ✅Succeeded
reference/5.1/Microsoft.PowerShell.Utility/Get-Runspace.md ✅Succeeded
reference/6/Microsoft.PowerShell.Utility/Get-FormatData.md ✅Succeeded
reference/docfx.json ✅Succeeded
reference/docs-conceptual/core-powershell/WSMan-Remoting-in-PowerShell-Core.md ✅Succeeded
reference/docs-conceptual/core-powershell/vscode/using-vscode.md ✅Succeeded
reference/docs-conceptual/docfx.json ✅Succeeded
reference/docs-conceptual/images/vscode.png ✅Succeeded
reference/docs-conceptual/setup/Installing-PowerShell-Core-on-Windows.md ✅Succeeded
reference/docs-conceptual/whats-new/What-s-New-in-PowerShell-Core-60.md ✅Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@saldana
Copy link
Contributor

saldana commented Jan 12, 2018

✅ Validation status: passed

File Status Preview URL Details
reference/5.1/Microsoft.PowerShell.Utility/Get-FormatData.md ✅Succeeded
reference/5.1/Microsoft.PowerShell.Utility/Get-Runspace.md ✅Succeeded
reference/6/Microsoft.PowerShell.Utility/Get-FormatData.md ✅Succeeded
reference/docfx.json ✅Succeeded
reference/docs-conceptual/core-powershell/WSMan-Remoting-in-PowerShell-Core.md ✅Succeeded
reference/docs-conceptual/setup/Installing-PowerShell-Core-on-Windows.md ✅Succeeded
reference/docs-conceptual/whats-new/What-s-New-in-PowerShell-Core-60.md ✅Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@sdwheeler sdwheeler merged commit 12dcead into live Jan 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.