Skip to content

new branch#6

Open
helium729 wants to merge 2 commits intomainfrom
bool-test
Open

new branch#6
helium729 wants to merge 2 commits intomainfrom
bool-test

Conversation

@helium729
Copy link
Owner

No description provided.

@helium729 helium729 requested a review from Copilot June 19, 2025 03:57
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 introduces support for boolean parameters in the parser. Key changes include adding a new add_bool method to the parser, implementing the parameter_bool class, and adding tests to verify its behavior.

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/test_parameter_bool.cc Adds tests to validate the boolean parameter functionality.
src/argparse/util.cc Registers the new BOOL parameter type in create_parameter.
src/argparse/parser.cc Implements the new add_bool method to handle boolean parameters.
src/argparse/parameter_bool.cc Provides the implementation for parameter_bool's logic.
include/argparse/util.h Updates header includes to reference parameter_bool.
include/argparse/parser.h Declares the new add_bool function.
include/argparse/parameter_bool.h Declares the parameter_bool class.
include/argparse/parameter.h Updates the parameter_type enum to include BOOL.

Co-authored-by: Copilot <175728472+Copilot@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.

1 participant