Skip to content

refactor: rename CMake project from NetworkSystem to network_system#916

Merged
kcenon merged 1 commit into
mainfrom
refactor/issue-914-rename-cmake-to-snake-case
Apr 3, 2026
Merged

refactor: rename CMake project from NetworkSystem to network_system#916
kcenon merged 1 commit into
mainfrom
refactor/issue-914-rename-cmake-to-snake-case

Conversation

@kcenon

@kcenon kcenon commented Apr 3, 2026

Copy link
Copy Markdown
Owner

Summary

Renames the CMake project from PascalCase (NetworkSystem) to snake_case (network_system) across the entire build system and documentation.

  • Rename project(NetworkSystem) to project(network_system) and all CMake targets
  • Rename 5 cmake module files from PascalCase to snake_case
  • Update config.cmake.in namespace to network_system::network_system
  • Remove legacy backward-compatibility variables (no aliases)
  • Update all test/sample/benchmark/integration CMakeLists.txt references
  • Update vcpkg port usage file and portfile
  • Update CMake examples in 18 documentation files
  • Update release workflow

Closes #914

What

Aspect Before After
Project name NetworkSystem network_system
Library target NetworkSystem network_system
CMake namespace network_system::NetworkSystem network_system::network_system
Module files NetworkSystemFeatures.cmake etc. network_system_features.cmake etc.

Why

Part of the kcenon ecosystem standardization (Phase 2). All CMake project names are being unified to snake_case. No backward-compatible aliases are provided - this is a clean break.

Where

  • Root CMakeLists.txt and 5 cmake module files (renamed)
  • cmake/network_system-config.cmake.in
  • 10 subdirectory CMakeLists.txt files (tests, samples, benchmarks, fuzz, examples, integration)
  • vcpkg-ports/kcenon-network-system/ (usage + portfile)
  • 18 documentation files (migration guides, quick start, build guides, troubleshooting, READMEs)
  • .github/workflows/release.yml

How

  1. Renamed cmake module files mechanically
  2. Updated all CMake target references in build files
  3. Updated documentation CMake examples (only CMake target/package refs, not C++ class names like NetworkSystemBridge)
  4. Removed legacy backward-compatibility variables from config template

Test plan

  • CI builds pass on all platforms (Linux, macOS, Windows)
  • All tests pass
  • find_package(network_system) works correctly
  • target_link_libraries(... network_system::network_system) resolves

Closes #914

- Rename project() from NetworkSystem to network_system
- Rename all CMake targets (add_library, target_link_libraries, etc.)
- Rename cmake module files to snake_case:
  NetworkSystemFeatures.cmake -> network_system_features.cmake
  NetworkSystemDependencies.cmake -> network_system_dependencies.cmake
  NetworkSystemCPack.cmake -> network_system_cpack.cmake
  NetworkSystemInstall.cmake -> network_system_install.cmake
  NetworkSystemIntegration.cmake -> network_system_integration.cmake
- Update config.cmake.in namespace to network_system::network_system
- Remove legacy backward-compatibility variables
- Update all test/sample/benchmark CMakeLists.txt references
- Update vcpkg port usage and portfile
- Update documentation CMake examples
- Update release workflow
- No backward-compatible aliases provided (clean break)
@github-actions

github-actions Bot commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

Performance Comparison

Base Branch Results

No base results

PR Branch Results

No PR results

@github-actions

github-actions Bot commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

Coverage Report

Metric Value
Line Coverage 62.0%
Target 75% (stretch: 80%)
Coverage Details

Full HTML report is available as a build artifact.

@kcenon kcenon merged commit 093691d into main Apr 3, 2026
43 checks passed
@kcenon kcenon deleted the refactor/issue-914-rename-cmake-to-snake-case branch April 3, 2026 01:41
kcenon added a commit that referenced this pull request Apr 13, 2026
…916)

Closes #914

- Rename project() from NetworkSystem to network_system
- Rename all CMake targets (add_library, target_link_libraries, etc.)
- Rename cmake module files to snake_case:
  NetworkSystemFeatures.cmake -> network_system_features.cmake
  NetworkSystemDependencies.cmake -> network_system_dependencies.cmake
  NetworkSystemCPack.cmake -> network_system_cpack.cmake
  NetworkSystemInstall.cmake -> network_system_install.cmake
  NetworkSystemIntegration.cmake -> network_system_integration.cmake
- Update config.cmake.in namespace to network_system::network_system
- Remove legacy backward-compatibility variables
- Update all test/sample/benchmark CMakeLists.txt references
- Update vcpkg port usage and portfile
- Update documentation CMake examples
- Update release workflow
- No backward-compatible aliases provided (clean break)
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.

refactor: rename CMake project from NetworkSystem to network_system

1 participant