-
Notifications
You must be signed in to change notification settings - Fork 6k
Updating ms.topic for AI readiness #46448
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
base: main
Are you sure you want to change the base?
Conversation
docfx.json
Outdated
"docs/framework/configure-apps/file-schema/**/**.md": "reference", | ||
"docs/framework/data/adonet/ef/language-reference/*-entity-sql.md": "language-reference", | ||
"docs/framework/data/adonet/ef/language-reference/*.md": "language-reference", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was the -entity-sql dropped here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To follow the pattern on line 275 and to prevent updates to individual files. But I can bring it back to the previous value if you prefer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it was done purposely that way because not all the files are actually reference type articles. Example: https://learn.microsoft.com/en-us/dotnet/framework/data/adonet/ef/language-reference/how-to-call-canonical-functions
docfx.json
Outdated
@@ -255,7 +255,10 @@ | |||
"_csharplang/**/*.md": "language-reference", | |||
"_csharpstandard/**/*.md": "language-reference", | |||
"_vblang/spec/*.md": "language-reference", | |||
"docs/azure/**/**.md": "conceptual", | |||
"docs/architecture/**/*.md": "concept-article", | |||
"docs/azure/**/**.md": "article", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not needed since it's the default, correct? (Here and in a few other places.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. I will remove any pattern that sets the ms.topic value to article in fileMetadata section.
@dotnet-policy-service agree company="Microsoft" |
Summary
Phase 1 of updating ms.topic as discussed with @BillWagner.
Internal previews
Toggle expand/collapse
Note
This table shows preview links for the 30 files with the most changes. For preview links for other files in this PR, select OpenPublishing.Build Details within checks.