Add translator comments for ambiguous resource strings in CodeAnalysisResources.resx#82080
Conversation
…Resources.resx Co-authored-by: 333fred <2371880+333fred@users.noreply.github.com>
|
@dotnet/roslyn-compiler for a very simple review |
Added comments to Assembly, Attribute, Constructor, Field, Method, Module, Parameter, Property, and TypeParameter to ensure consistency and clarity for all attribute target resources. Co-authored-by: 333fred <2371880+333fred@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
| </data> | ||
| <data name="Constructor" xml:space="preserve"> | ||
| <value>constructor</value> | ||
| <comment>Refers to a type constructor in programming (noun)</comment> |
There was a problem hiding this comment.
@copilot we don't need these types of comments where there is no confusion. #Closed
There was a problem hiding this comment.
Removed unnecessary comments from non-ambiguous resources (Assembly, Constructor, Field, Method, Module, Parameter, Property, TypeParameter). Kept comments only for resources with clear verb/noun ambiguity. Commit: 64f0a2d
Removed comments from Assembly, Constructor, Field, Method, Module, Parameter, Property, and TypeParameter as these don't have verb/noun ambiguity. Kept comments only for resources with clear ambiguity: Return1, Class1, Delegate1, Event1, Interface1, Struct1, Enum1, and Attribute. Co-authored-by: 333fred <2371880+333fred@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@dotnet/roslyn-compiler for a second review. |
|
@copilot address merge conflicts. |
Original prompt
This pull request was created from Copilot chat.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.