-
Notifications
You must be signed in to change notification settings - Fork 6.1k
Prevent tags from loc #50214
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
Prevent tags from loc #50214
Conversation
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 contributes to issue #50193 by preventing certain tags from being sent to localization. The changes primarily focus on:
- Removing trailing spaces after YAML frontmatter colons (e.g.,
helpviewer_keywords:anddev_langs:) - Converting bold formatting (
**term**) to code formatting (`term`) for technical terms, keywords, function names, and type names - Removing trailing spaces throughout the documentation
These changes align with the Markdown writing style guidelines to use backticks for code, technical types, and non-localizable text rather than bold formatting.
Reviewed changes
Copilot reviewed 24 out of 24 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| specifying-an-entry-point.md | Removed trailing spaces; converted technical keywords (Function, Declare, MessageBox, Alias, Auto, EntryPoint) from bold to backticks |
| specifying-a-character-set.md | Removed trailing spaces; converted technical terms (MessageBox, MessageBoxA, MessageBoxW, ExactSpelling, CharSet, etc.) from bold to backticks |
| registering-assemblies-with-com.md | Removed trailing spaces; converted DllGetClassObject from bold to backticks |
| platform-invoke-examples.md | Removed trailing spaces; converted MessageBox and Auto from bold to backticks |
| passing-structures.md | Removed trailing spaces; converted LayoutKind, ByRef, PtInRect, GetSystemTime from bold to backticks; inconsistent formatting on line 22 |
| msgbox-sample.md | Removed trailing spaces; converted MessageBox, EntryPoint, ExactSpelling from bold to backticks; inconsistent formatting on line 29 |
| marshalling-different-types-of-arrays.md | Removed trailing spaces; converted technical terms (TestArrayOfInts, InAttribute, OutAttribute, MYPOINT) from bold to backticks; inconsistent formatting on line 84 |
| marshalling-classes-structures-and-unions.md | Removed trailing spaces; converted multiple technical terms (TestStructInStruct, TestArrayInStruct, IntPtr, ref, MarshalAsAttribute) from bold to backticks; inconsistent formatting on lines 159 and 187 |
| identifying-functions-in-dlls.md | Removed trailing spaces; converted MessageBox, MessageBoxA, MessageBoxW from bold to backticks |
| how-to-reference-net-types-from-com.md | Removed trailing spaces; converted #import and raw_interfaces_only from bold to backticks |
| how-to-implement-callback-functions.md | Removed trailing spaces; converted EnumWindows, lpEnumFunc, Func, CallBack, hwnd, lparam, true from bold to backticks |
| exposing-dotnet-components-to-com.md | Removed trailing spaces; converted CoCreateInstance from bold to backticks |
| default-marshalling-for-objects.md | Removed trailing spaces; converted multiple technical terms (Object, UnmanagedType, IDispatch, IUnknown, etc.) from bold to backticks |
| default-marshalling-for-arrays.md | Removed trailing spaces; converted technical terms (ELEMENT_TYPE_ARRAY, SAFEARRAY, BSTR, etc.) from bold to backticks; fixed table alignment |
| default-marshalling-behavior.md | Removed trailing spaces; converted technical terms (BSTR, String, IntPtr, CoTaskMemFree, System, etc.) from bold to backticks |
| copying-and-pinning.md | Removed trailing spaces; converted InAttribute, OutAttribute, BSTR, StringBuilder from bold to backticks |
| consuming-unmanaged-dll-functions.md | Removed trailing spaces; converted Declare, Function, Lib, DllImportAttribute, static, extern from bold to backticks |
| com-interop-sample-net-client-and-com-server.md | Removed trailing spaces; converted Loan from bold to backticks |
| callback-functions.md | Removed trailing spaces; converted EnumWindows from bold to backticks |
| security-trust-levels-in-accessing-resources.md | Removed trailing spaces; converted AllowPartiallyTrustedCallers from bold to backticks |
| optimization-spc-and-promotable-spn.md | Removed trailing spaces; converted TransactionCommitted from bold to backticks |
| implementing-a-resource-manager.md | Removed trailing spaces; converted Enlist from bold to backticks |
| enlisting-resources-as-participants-in-a-transaction.md | Removed trailing spaces; converted Enlist from bold to backticks |
| diagnostic-traces.md | Removed trailing spaces; converted add, initializeData, switchValue, TraceRecord from bold to backticks |
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
docs/framework/interop/marshalling-different-types-of-arrays.md
Outdated
Show resolved
Hide resolved
docs/framework/interop/marshalling-classes-structures-and-unions.md
Outdated
Show resolved
Hide resolved
docs/framework/interop/marshalling-classes-structures-and-unions.md
Outdated
Show resolved
Hide resolved
BillWagner
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.
This also LGTM. Once again, it looks like some of the Copilot review feedback is reasonable.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributes to #50193.
Internal previews
Toggle expand/collapse