Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CPP-AP: version 1.1 #46

Merged
merged 5 commits into from
May 16, 2024
Merged

CPP-AP: version 1.1 #46

merged 5 commits into from
May 16, 2024

Conversation

SpectraL519
Copy link
Owner

@SpectraL519 SpectraL519 commented May 16, 2024

  • Introduced CMake integration
  • Formatting scripts improvement
  • Small code refactor including:
    • argument_name class refactor
    • Added aliases for default positional and optional arguments discriminator structures
    • Utilizing the std::ranges library for operations on collections
  • Cleaned the commit history

- Reorganized the project folder structure to use more descriptive names
- Improved formatting scripts to enumerate the formatted files and run clang-format check with the `--check` flag
- Modified the `install_clang_format_17.sh` script to install both `clang-17` and `clang-format-17` toolchains
- Added a format checking workflow
- Modified the `g++` and `clang++` workflows to properly set both C and C++ comilers with CMake
- Added the change log
- Renamed the `argument_name` structure members:
  - `name` -> `primary`
  - `short_name` -> `secondary`
- Added the `argument_name::match` functions
- Aligned the code and unit tests to use the new names and functions
- Fixed some typos and incorrect examples in the readme file
- Added markdown alerts in the readme file
- Renamed and refactored the `ap::action::check_file_exists_action()` function
- Improved the istream operator in example program: verbosity
- Added the root `CMakeLists.txt` and `cmake/cpp-ap-config.cmake.in` files to enable CMake integration for the library
- Removed the example directory and added the `cpp-ap-demo` submodule 
- Modified the README file and GitHub workflows to account for the changes
Copy link
Owner Author

@SpectraL519 SpectraL519 left a comment

Choose a reason for hiding this comment

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

:)

- Introduced CMake integration
- Formatting scripts improvement
- Small code refactor including:
    - `argument_name` class refactor
    - Added aliases for default positional and optional arguments discriminator structures
    - Utilizing the `std::ranges` library for operations on collections
- Cleaned the commit history
@SpectraL519 SpectraL519 merged commit 3f0f32e into master May 16, 2024
3 checks passed
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.

1 participant