Skip to content

refactor: reorganize InternalsVisibleTo directives in AssemblyInfo.cs files#2387

Merged
0xFA11 merged 1 commit intodevelopfrom
refactor/assemblyinfo
Jan 18, 2023
Merged

refactor: reorganize InternalsVisibleTo directives in AssemblyInfo.cs files#2387
0xFA11 merged 1 commit intodevelopfrom
refactor/assemblyinfo

Conversation

@0xFA11
Copy link
Copy Markdown
Contributor

@0xFA11 0xFA11 commented Jan 18, 2023

By using UNITY_INCLUDE_TESTS, UNITY_EDITOR, MULTIPLAYER_TOOLS and other macros, we're exposing internals of asmdefs more carefully with much more limited scope/exposure.

@0xFA11 0xFA11 marked this pull request as ready for review January 18, 2023 13:40
@0xFA11 0xFA11 enabled auto-merge (squash) January 18, 2023 13:42
Copy link
Copy Markdown
Member

@NoelStephensUnity NoelStephensUnity left a comment

Choose a reason for hiding this comment

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

Awesome!

@0xFA11 0xFA11 merged commit 3d2c266 into develop Jan 18, 2023
@0xFA11 0xFA11 deleted the refactor/assemblyinfo branch January 18, 2023 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants