Skip to content

Fix a few issues in markdown help files #5574

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 6 commits into from
Feb 22, 2018

Conversation

cormacpayne
Copy link
Member

@cormacpayne cormacpayne commented Feb 17, 2018

Description

  • Add missing entries under INPUTS header in markdown files
  • Fix error in the header of PowerBI markdown help files
  • Add missing help URL in the header of markdown help files
  • Add check in CI to validate that all markdown files have a corresponding help URL in their header

Checklist

maddieclayton
maddieclayton previously approved these changes Feb 17, 2018
@maddieclayton
Copy link
Contributor

@MiYanni
Copy link
Contributor

MiYanni commented Feb 20, 2018

@cormacpayne These guys need fixing:

Get-AzureStorageServiceProperty.md,No entry was found for the online version field in the header. The corresponding URL should be the following: https://docs.microsoft.com/en-us/powershell/module/azure.storage/get-azurestorageserviceproperty
Update-AzureStorageServiceProperty.md,No entry was found for the online version field in the header. The corresponding URL should be the following: https://docs.microsoft.com/en-us/powershell/module/azure.storage/update-azurestorageserviceproperty

@cormacpayne
Copy link
Member Author

@MiYanni I was just testing to make sure the script worked 😉

@cormacpayne
Copy link
Member Author

@cormacpayne cormacpayne changed the base branch from preview to release-2018-02-23 February 21, 2018 23:33
@cormacpayne
Copy link
Member Author

@maddieclayton maddieclayton removed their assignment Feb 21, 2018
@cormacpayne cormacpayne merged commit 7cb99d5 into Azure:release-2018-02-23 Feb 22, 2018
@cormacpayne cormacpayne deleted the docs-fix branch May 2, 2018 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants