Skip to content

Conversation

@baronfel
Copy link
Member

@baronfel baronfel commented Oct 8, 2025

Since Strong Naming has no impact on modern .NET Runtimes, limit the scope of guidance to .NET Framework and .NET Standard


Internal previews

📄 File 🔗 Preview link
docs/standard/library-guidance/strong-naming.md Strong naming

…licable platforms

Since Strong Naming has no impact on modern .NET Runtimes, limit the scope of guidance to .NET Framework and .NET Standard
Copilot AI review requested due to automatic review settings October 8, 2025 15:15
@baronfel baronfel requested review from a team, IEvangelist and JamesNK as code owners October 8, 2025 15:15
@dotnetrepoman dotnetrepoman bot added this to the October 2025 milestone Oct 8, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates strong naming guidance for .NET libraries to clarify that it only applies to .NET Framework and .NET Standard, not modern .NET runtimes. This change helps developers understand when strong naming is actually necessary.

  • Modified the primary recommendation to specify targeting .NET Framework or .NET Standard
  • Added explanatory note that strong naming has no impact on modern .NET runtimes

Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a suggestion for consideration. Thanks for adding this!

baronfel and others added 3 commits October 8, 2025 10:40
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
If multitargeting, strong naming should be done for all TFMs so that InternalsVisibleTo attributes can remain consistent.
@IEvangelist IEvangelist enabled auto-merge (squash) October 13, 2025 14:36
@IEvangelist IEvangelist merged commit 7aadd0f into main Oct 13, 2025
10 checks passed
@IEvangelist IEvangelist deleted the baronfel-patch-1 branch October 13, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants