Skip to content

Readme Content Verification#3

Merged
scbedd merged 1 commit into
Azure:masterfrom
scbedd:readme-content-verification
Feb 26, 2019
Merged

Readme Content Verification#3
scbedd merged 1 commit into
Azure:masterfrom
scbedd:readme-content-verification

Conversation

@scbedd

@scbedd scbedd commented Feb 22, 2019

Copy link
Copy Markdown
Member
  • Cleaning up namespace to just pieces we want public. Getting rid of import *
  • Adding verification of a readme content.
    • Currently works by looking at the section headers.
    • Supports regex matching of said section headers
    • Works with the omitted_paths configuration so we will ignore readmes as expected

Before merge, I will update the readme with additional details on example usage of the readme verification.

Issue
@Azure/azure-sdk-eng

Comment thread packages/python-packages/doc-warden/warden/WardenConfiguration.py Outdated
…port ordering cleanup and whitespace cleanup. updating all copyright comments with the appropriate boilerplate comment from opensource. readme updates for content verification usage
@scbedd scbedd force-pushed the readme-content-verification branch from 0739737 to 19b1712 Compare February 26, 2019 17:16
@scbedd scbedd merged commit a2ff91a into Azure:master Feb 26, 2019
@mmacy

mmacy commented Mar 1, 2019

Copy link
Copy Markdown
Contributor

@scbedd Hey Scott, just wanted to verify the tool's validation is based on the latest README format defined in README-TEMPLATE.md.

arpanlaha added a commit to arpanlaha/azure-sdk-tools that referenced this pull request Jun 14, 2019
mitchdenny added a commit to mitchdenny/azure-sdk-tools that referenced this pull request Apr 19, 2020
sima-zhu pushed a commit to sima-zhu/azure-sdk-tools that referenced this pull request Dec 3, 2020
Simple CMake in AzureDevOps
wanlwanl added a commit to albertxavier100/azure-sdk-tools that referenced this pull request Jun 6, 2025
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.

4 participants