Skip to content

MCP test #680

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

Closed
wants to merge 2 commits into from
Closed

MCP test #680

wants to merge 2 commits into from

Conversation

OwenRichards1
Copy link
Contributor

Using local MCP to update API ref documentation

@OwenRichards1 OwenRichards1 requested a review from a team as a code owner July 9, 2025 14:35
Copy link
Contributor

Learn Build status updates of commit 4c09ce8:

⚠️ Validation status: warnings

File Status Preview URL Details
msal-dotnet-articles/getting-started/authentication-decision-tree.md ⚠️Warning View Details
msal-dotnet-articles/getting-started/quick-start-guide.md ⚠️Warning View Details
.gitignore ✅Succeeded
msal-dotnet-articles/advanced/exceptions/practical-error-handling.md ✅Succeeded View
msal-dotnet-articles/getting-started/best-practices.md ✅Succeeded View
msal-dotnet-articles/getting-started/code-examples.md ✅Succeeded View
msal-dotnet-articles/how-to/migration-troubleshooting.md ✅Succeeded View
msal-dotnet-articles/TOC.yml ✅Succeeded View

msal-dotnet-articles/getting-started/authentication-decision-tree.md

  • Line 384, Column 23: [Warning: file-not-found - See documentation] Invalid file link: 'msal-net-migration.md'.

msal-dotnet-articles/getting-started/quick-start-guide.md

  • Line 169, Column 3: [Warning: file-not-found - See documentation] Invalid file link: '../../../dotnet/api/overview/index.md'.

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.

Copy link
Contributor

Learn Build status updates of commit b2f4976:

⚠️ Validation status: warnings

File Status Preview URL Details
msal-dotnet-articles/code-samples/modern-confidential-client.md ⚠️Warning View Details
msal-dotnet-articles/code-samples/modern-desktop-wam.md ⚠️Warning View Details
msal-dotnet-articles/code-samples/ModernConfidentialClient/README.md ⚠️Warning View Details
msal-dotnet-articles/code-samples/ModernDesktopWAM/README.md ⚠️Warning View Details
msal-dotnet-articles/code-samples/README.md ⚠️Warning View Details
msal-dotnet-articles/getting-started/authentication-decision-tree.md ⚠️Warning View Details
msal-dotnet-articles/getting-started/quick-start-guide.md ⚠️Warning View Details
.gitignore ✅Succeeded
msal-dotnet-articles/advanced/exceptions/practical-error-handling.md ✅Succeeded View
msal-dotnet-articles/code-samples/ModernConfidentialClient/appsettings.json ✅Succeeded
msal-dotnet-articles/code-samples/ModernConfidentialClient/ModernConfidentialClient.csproj ✅Succeeded
msal-dotnet-articles/code-samples/ModernConfidentialClient/Program.cs ✅Succeeded
msal-dotnet-articles/code-samples/ModernDesktopWAM/appsettings.json ✅Succeeded
msal-dotnet-articles/code-samples/ModernDesktopWAM/ModernDesktopWAM.csproj ✅Succeeded
msal-dotnet-articles/code-samples/ModernDesktopWAM/Program.cs ✅Succeeded
msal-dotnet-articles/getting-started/best-practices.md ✅Succeeded View
msal-dotnet-articles/getting-started/code-examples.md ✅Succeeded View
msal-dotnet-articles/how-to/migration-troubleshooting.md ✅Succeeded View
msal-dotnet-articles/TOC.yml ✅Succeeded View

msal-dotnet-articles/code-samples/modern-confidential-client.md

  • Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
  • Line 1, Column 1: [Warning: duplicate-h1s - See documentation] H1 'Modern Confidential Client Application with Azure Key Vault' is duplicated with other articles: 'code-samples/modern-confidential-client.md, code-samples/ModernConfidentialClient/README.md'. First level headings must be unique within a docset.
  • Line 1, Column 1: [Warning: duplicate-titles - See documentation] Title 'Modern Confidential Client Application with Azure Key Vault - Microsoft Authentication Library for .NET' is duplicated in: 'code-samples/modern-confidential-client.md, code-samples/ModernConfidentialClient/README.md'. Titles must be unique within a docset.
  • Line 1, Column 1: [Suggestion: ms-date-missing - See documentation] Missing required attribute: 'ms.date'.
  • Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
  • Line 578, Column 3: [Suggestion: learn-site-link-broken - See documentation] Link 'https://learn.microsoft.com/azure/active-directory/develop/msal-net-confidential-client' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 578, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/azure/active-directory/develop/msal-net-confidential-client' will be broken in isolated environments. Replace with a relative link.
  • Line 579, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/azure/key-vault/general/overview' will be broken in isolated environments. Replace with a relative link.
  • Line 580, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/azure/active-directory/managed-identities-azure-resources/overview' will be broken in isolated environments. Replace with a relative link.
  • Line 581, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/azure/active-directory/develop/v2-oauth2-client-creds-grant-flow' will be broken in isolated environments. Replace with a relative link.
  • Line 582, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/azure/active-directory/develop/v2-oauth2-on-behalf-of-flow' will be broken in isolated environments. Replace with a relative link.

msal-dotnet-articles/code-samples/modern-desktop-wam.md

  • Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
  • Line 1, Column 1: [Warning: duplicate-h1s - See documentation] H1 'Modern Desktop Application with WAM (Windows Authentication Manager)' is duplicated with other articles: 'code-samples/modern-desktop-wam.md, code-samples/ModernDesktopWAM/README.md'. First level headings must be unique within a docset.
  • Line 1, Column 1: [Warning: duplicate-titles - See documentation] Title 'Modern Desktop Application with WAM (Windows Authentication Manager) - Microsoft Authentication Library for .NET' is duplicated in: 'code-samples/modern-desktop-wam.md, code-samples/ModernDesktopWAM/README.md'. Titles must be unique within a docset.
  • Line 0, Column 0: [Suggestion: filename-incomplete - See documentation] The filename contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.
  • Line 1, Column 1: [Suggestion: ms-date-missing - See documentation] Missing required attribute: 'ms.date'.
  • Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
  • Line 336, Column 3: [Suggestion: learn-site-link-broken - See documentation] Link 'https://learn.microsoft.com/azure/active-directory/develop/msal-net-overview' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 336, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/azure/active-directory/develop/msal-net-overview' will be broken in isolated environments. Replace with a relative link.
  • Line 337, Column 3: [Suggestion: learn-site-link-broken - See documentation] Link 'https://learn.microsoft.com/azure/active-directory/develop/msal-net-wam' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 337, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/azure/active-directory/develop/msal-net-wam' will be broken in isolated environments. Replace with a relative link.
  • Line 338, Column 3: [Suggestion: learn-site-link-broken - See documentation] Link 'https://learn.microsoft.com/azure/active-directory/develop/msal-net-regional-endpoints' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 338, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/azure/active-directory/develop/msal-net-regional-endpoints' will be broken in isolated environments. Replace with a relative link.

msal-dotnet-articles/code-samples/ModernConfidentialClient/README.md

  • Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
  • Line 1, Column 1: [Warning: duplicate-h1s - See documentation] H1 'Modern Confidential Client Application with Azure Key Vault' is duplicated with other articles: 'code-samples/modern-confidential-client.md, code-samples/ModernConfidentialClient/README.md'. First level headings must be unique within a docset.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
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.

@OwenRichards1 OwenRichards1 marked this pull request as draft July 16, 2025 12:36
@OwenRichards1 OwenRichards1 deleted the mcp-server branch July 21, 2025 16:30
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