Skip to content
This repository was archived by the owner on Feb 5, 2025. It is now read-only.

Adopt namespace naming guidelines - part 1 #1384

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

mlw
Copy link
Contributor

@mlw mlw commented Jul 4, 2024

The style guide prefers that a projects do not overly nest namespace names, and note that unless necessary, a single top-level namespace per project is the most effective strategy to limit exposure to collisions.

To keep PR size reasonable, part 1 only changes things in santa::common and santa::santad::data_layer. Other namespaces will be converted in a future PR.

References:

@mlw mlw added the cleanup label Jul 4, 2024
@mlw mlw added this to the 2024.7 milestone Jul 4, 2024
@mlw mlw requested a review from a team as a code owner July 4, 2024 18:28
@mlw mlw force-pushed the namespace-simplification-pt1 branch from f74e0fa to 7689d5a Compare July 4, 2024 18:28
@mlw mlw merged commit 73c1885 into google:main Jul 5, 2024
10 checks passed
@mlw mlw modified the milestones: 2024.7, 2024.6 Jul 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants