Skip to content

Conversation

@Binyang2014
Copy link
Contributor

No description provided.

Binyang2014 and others added 30 commits January 15, 2026 07:38
…d style consistency (#725)

- [x] Fix license text: "MIT license" → "MIT License" in multiple files
- [x] Rename files with "_2" suffix and update references
- [x] Add missing license headers
- [x] Fix header guards to follow MSCCLPP_EXT_<FILE_NAME>_HPP_ format
- [x] Fix enum naming consistency
  - [x] CommResult enum to CamelCase
- [x] CollectiveBufferMode enum to CamelCase (Any, InPlace, OutOfPlace)
  - [x] AlgorithmType enum to CamelCase (Native, DSL)
- [x] Fix comment in src/core/gpu_utils.cc
- [x] Fix LogSubsys::COUNT case in src/core/include/logger.hpp
  - [x] Add explanatory comment
  - [x] Add [[fallthrough]] attribute
- [x] Apply clang-format
- [x] Remove _codeql_detected_source_root file and add to .gitignore
- [x] Update documentation paths
  - [x] Fix NCCL library paths: build/apps/nccl/ → build/lib/
  - [x] Fix test binary paths: ./test/ → ./bin/

<!-- START COPILOT CODING AGENT TIPS -->
---

💬 We'd love your input! Share your thoughts on Copilot coding agent in
our [2 minute survey](https://gh.io/copilot-coding-agent-survey).

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: chhwang <8018170+chhwang@users.noreply.github.com>
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.

3 participants