Skip to content

Update guidance for CA rules docs#1175

Open
gewarren wants to merge 5 commits intoMicrosoftDocs:mainfrom
gewarren:ca-rules-guidance
Open

Update guidance for CA rules docs#1175
gewarren wants to merge 5 commits intoMicrosoftDocs:mainfrom
gewarren:ca-rules-guidance

Conversation

@gewarren
Copy link
Member

Contributor guide pull request

Thanks for contributing to the Microsoft Learn contributor guide. Please read these instructions for processing your pull request (PR). Once you've read this text, delete it and write a brief description of the changes you're proposing.

Quality control

  • 1. Successful build with no warnings or errors: Review the build status to make sure all checks are green (Succeeded).

  • 2. #Sign-off: Once the PR is finalized and ready to be merged, indicate so by typing #sign-off in a new comment in the PR. Signing off means the document is ready for review and can be published at any time.

Merge and publish

  • All PRs to this repository are manually reviewed and merged.
  • Once all feedback on the PR is addressed, we'll merge the PR into the main branch.

Need help?

If you have an open PR, tag the PR reviewers in a comment with your question: @mikekinsman, @jehchow

@learn-build-service-prod
Copy link

Learn Build status updates of commit d852579:

💡 Validation status: suggestions

File Status Preview URL Details
.openpublishing.redirection.json 💡Suggestion View Details
Contribute/content/dotnet/code-analysis-rules.md ✅Succeeded View
Contribute/content/dotnet/dotnet-contribute-code-analysis.md ✅Succeeded n/a (file deleted or renamed)

.openpublishing.redirection.json

  • Line 16, Column 68: [Suggestion: redirect-url-invalid - See documentation] Can't redirect document ID for redirected file 'content/dotnet/dotnet-contribute-code-analysis.md' because redirect URL '/contribute/dotnet/code-analysis-rules' is invalid or is in a different docset. Specify a redirect_url in the same docset, or set redirect_document_id to false in .openpublishing.redirection.json.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link

Learn Build status updates of commit 8744e94:

💡 Validation status: suggestions

File Status Preview URL Details
.openpublishing.redirection.json 💡Suggestion View Details
Contribute/content/dotnet/code-analysis-rules.md ✅Succeeded View
Contribute/content/dotnet/dotnet-contribute-code-analysis.md ✅Succeeded n/a (file deleted or renamed)

.openpublishing.redirection.json

  • Line 16, Column 68: [Suggestion: redirect-url-invalid - See documentation] Can't redirect document ID for redirected file 'content/dotnet/dotnet-contribute-code-analysis.md' because redirect URL '/contribute/dotnet/code-analysis-rules' is invalid or is in a different docset. Specify a redirect_url in the same docset, or set redirect_document_id to false in .openpublishing.redirection.json.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link

Learn Build status updates of commit 6b76872:

💡 Validation status: suggestions

File Status Preview URL Details
Contribute/index.yml 💡Suggestion View Details
.openpublishing.redirection.json ✅Succeeded View
Contribute/content/dotnet/code-analysis-rules.md ✅Succeeded View
Contribute/content/dotnet/dotnet-contribute-code-analysis.md ✅Succeeded n/a (file deleted or renamed)
Contribute/content/TOC.yml ✅Succeeded View

Contribute/index.yml

  • Line 55, Column 71: [Suggestion: learn-site-link-broken - See documentation] Link 'https://learn.microsoft.com/answers/questions/5722678/what-are-the-different-between-developer-command-p' points to a learn site page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link

Learn Build status updates of commit 7dbb327:

✅ Validation status: passed

File Status Preview URL Details
.openpublishing.redirection.json ✅Succeeded View
Contribute/content/dotnet/code-analysis-rules.md ✅Succeeded View
Contribute/content/dotnet/dotnet-contribute-code-analysis.md ✅Succeeded n/a (file deleted or renamed)
Contribute/content/TOC.yml ✅Succeeded View
Contribute/index.yml ✅Succeeded View

For more details, please refer to the build report.

@gewarren
Copy link
Member Author

#sign-off

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.

1 participant