Skip to content

Conversation

@euanh
Copy link
Collaborator

@euanh euanh commented Feb 14, 2025

Motivation

In many places we import a whole module but only use one or two structs or classes. In these cases, importing just the parts we need makes dependencies clearer.

Modifications

  • Where only one definition is used, constrain the import statement to only that definition.
  • Remove a couple of Foundation imports which were completely unused.

Result

No functional change. Dependencies are clearer.

Test Plan

Tests continue to pass.

@euanh euanh added the semver/none No version bump required. label Feb 14, 2025
@euanh euanh merged commit 7f92276 into apple:main Feb 14, 2025
18 checks passed
@euanh euanh deleted the broad-imports branch February 14, 2025 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant