-
Notifications
You must be signed in to change notification settings - Fork 6.1k
Add version history include to the keywords folder #51273
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
This one had a lot of changes, including moving snippets, so let's build the draft PR now.
More moving snippet files, more freshness.
0675152 to
a9d85ba
Compare
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.
Pull request overview
This PR updates numerous C# keyword documentation files to add version history includes, modernize formatting, move code snippets to separate files, and apply a freshness pass across the keywords folder. The changes address issue #50891, which requested adding prominent links to version history for C# features.
Changes:
- Added version history includes (
[!INCLUDE[csharp-version-note](../includes/initial-version.md)]) to 50+ keyword documentation files - Updated
ms.datefields to 01/21/2026 or 01/22/2026 across all modified files - Moved inline code snippets to separate
.csfiles in thesnippetssubdirectory with proper snippet IDs - Modernized documentation style following Microsoft's voice and tone guidelines (active voice, contractions, "you" perspective)
- Restructured list formatting from tables to bullet lists in several files for improved readability
Reviewed changes
Copilot reviewed 82 out of 102 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| Multiple keyword .md files | Added version history include, updated dates, improved writing style |
| snippets/*.cs files | New code snippet files extracted from inline code |
| Project files (.csproj) | Deleted obsolete project files, updated main keywords.csproj |
| implicitly-typed-local-variables.md | Replaced snippet reference with inline code |
| Various deleted files | Removed old snippet infrastructure and project files |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
The snippets 5000 error is a false positive. Two project files were removed when the code moved to the keywords project. |
meaghanlewis
left a comment
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.
These changes LGTM.
Fixes #50891
Finish adding the include file in all files in the keywords folder.
This folder had several files that haven't been updated in years. That includes poor formatting, stale snippets, and more changes. There are more commits in this PR, and many more files. It may be easier to review commit-by-commit. Also, several formatting changes, so hide whitespace differences.
Overall, this PR did the following:
staticmembers in interfaces and parameterless constructors instructtypes.)For reviewers, the articles that had been updated recently have minimal changes. Those reference files that haven't needed updates since 2017 had many more edits.
Commits:
abstractkeyword page. Move snippets and freshness pass.group.null.Summary
Describe your changes here.
Fixes #Issue_Number (if available)
Internal previews
Toggle expand/collapse
eventkeyword (C# reference)refkeywordusingdirectiveNote
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.