Skip to content

Migreene patch1 #1205

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 12 commits into from
May 15, 2017
Merged

Migreene patch1 #1205

merged 12 commits into from
May 15, 2017

Conversation

mgreenegit
Copy link
Contributor

This change impacts only 1 file - related to offline configuration of package management. Based on customer feedback over Twitter. It is a fairly significant overhaul of that file content.

@mgreenegit mgreenegit added the in-progress Tag - issue is being worked on by someone label May 12, 2017
@saldana
Copy link
Contributor

saldana commented May 12, 2017

✅ Validation status: passed

File Status Preview URL Details
gallery/psget/repository/bootstrapping_nuget_proivder_and_exe.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.

@@ -6,36 +6,41 @@ author: jpjofre
ms.prod: powershell
keywords: powershell,cmdlet,gallery
ms.date: 2016-10-14
contributor: manikb
contributor: manikb, mgreenegit
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you don't need to change this. it just identifies primary author for internal metrics. also, it's internal alias and not github alias :)

Copy link
Contributor Author

@mgreenegit mgreenegit May 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@@ -6,36 +6,41 @@ author: jpjofre
ms.prod: powershell
keywords: powershell,cmdlet,gallery
ms.date: 2016-10-14
contributor: manikb
contributor: manikb, mgreenegit
title: bootstrapping_nuget_proivder_and_exe
Copy link
Contributor

@zjalexander zjalexander May 12, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, this title should not have underscores. This gets picked up as title tag metadata by search engines. Do you mind changing this as well?
EDIT: wait, i just looked it up and now i'm confused again. ignore this comment.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DEFINITIVE ANSWER:
Change the title.
image

Field: Title. Description: Title of the article. Purpose: (SEO) This is the title that gets displayed on a search engine results page. Only the first 65 characters will be displayed, so keep it brief. Use your main SEO keyword for this topic here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed


## When NuGet provider is not available
*Note - Starting with version 6, the NuGet provider is included in the installation of PowerShell.*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Proper way to do a note:

> **Note**:

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@zjalexander
Copy link
Contributor

fyi updated my note about the TITLE : metadata because I am deeply, deeply confused about OPS.

@zjalexander
Copy link
Contributor

oh, the title stuff is a widespread issue, reopened #1188 to track

@saldana
Copy link
Contributor

saldana commented May 13, 2017

✅ Validation status: passed

File Status Preview URL Details
gallery/psget/repository/bootstrapping_nuget_proivder_and_exe.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 May 15, 2017

✅ Validation status: passed

File Status Preview URL Details
gallery/psget/repository/bootstrapping_nuget_proivder_and_exe.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.

@zjalexander zjalexander merged commit 0205374 into staging May 15, 2017
@zjalexander
Copy link
Contributor

thanks!

@andyleejordan andyleejordan removed the in-progress Tag - issue is being worked on by someone label May 15, 2017
@saldana
Copy link
Contributor

saldana commented May 15, 2017

✅ Validation status: passed

File Status Preview URL Details
gallery/psget/repository/bootstrapping_nuget_proivder_and_exe.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.

@zjalexander zjalexander deleted the migreene-patch1 branch August 9, 2017 21:09
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.

5 participants