Skip to content

Conversation

@Certseeds
Copy link
Owner

部分描述见 #36 , 升级默认编译环境到 ubuntu24.04, gcc13, catch2-3.10.0

Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
finish pure-eight-queen problem

Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
and update compiler to gcc11

Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
…ubmit

Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
Signed-off-by: Certseeds <51754303+Certseeds@users.noreply.github.com>
@Certseeds Certseeds self-assigned this Sep 20, 2025
Copilot AI review requested due to automatic review settings September 20, 2025 17:09
@Certseeds Certseeds added the enhancement New feature or request label Sep 20, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements version 0.12.0 with significant infrastructure changes including upgrading the default compilation environment to Ubuntu 24.04, GCC 13, and Catch2-3.10.0. The primary change is a major refactoring from a static function-based approach to a class-based Solution pattern for LeetCode algorithm implementations.

Key changes:

  • Refactored all LeetCode implementations from static functions in structs to Solution classes
  • Converted test header files (.hpp) to test implementation files (.cpp)
  • Updated include guard naming convention from CS203_DSAA_TEMPLATE to ALGORITHM_TEMPLATE
  • Updated copyright headers to SPDX format with 2022-2025 date ranges

Reviewed Changes

Copilot reviewed 277 out of 1177 changed files in this pull request and generated no comments.

File Description
algorithm/array/leetcode_*_test.cpp Converted from .hpp to .cpp files with Solution class instantiation
algorithm/array/leetcode_*.cpp Refactored from static struct functions to Solution class methods
Various deleted .hpp files Removed old header-based test files in favor of .cpp implementation
Files not reviewed (22)
  • .idea/.gitattributes: Language not supported
  • .idea/.gitignore: Language not supported
  • .idea/codeStyles/.gitignore: Language not supported
  • .idea/codeStyles/Project.xml: Language not supported
  • .idea/codeStyles/codeStyleConfig.xml: Language not supported
  • .idea/encodings.xml: Language not supported
  • .idea/fileTemplates/includes/.gitignore: Language not supported
  • .idea/fileTemplates/includes/FILE_HEAD_CPP.h: Language not supported
  • .idea/fileTemplates/includes/FILE_HEAD_PY.py: Language not supported
  • .idea/fileTemplates/includes/LICENSE_AGPL_CPP.h: Language not supported
  • .idea/fileTemplates/includes/LICENSE_AGPL_PY.py: Language not supported
  • .idea/fileTemplates/internal/.gitignore: Language not supported
  • .idea/fileTemplates/internal/C Header File.h: Language not supported
  • .idea/fileTemplates/internal/C Source File.c: Language not supported
  • .idea/fileTemplates/internal/C++ Class Header.h: Language not supported
  • .idea/fileTemplates/internal/C++ Class.cc: Language not supported
  • .idea/fileTemplates/internal/Python Script.py: Language not supported
  • .idea/fileTemplates/internal/Python Unit Test.py: Language not supported
  • .idea/fileTemplates/j2ee/.gitignore: Language not supported
  • .idea/fileTemplates/j2ee/cpp.executable.CMakeLists.txt: Language not supported
  • .idea/fileTemplates/j2ee/cpp.library.CMakeLists.txt: Language not supported
  • .idea/inspectionProfiles/Project_Default.xml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Certseeds Certseeds merged commit 905822a into master Sep 20, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Development

Successfully merging this pull request may close these issues.

2 participants