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

Clean CLI integration #22

Merged
merged 5 commits into from
Nov 27, 2024
Merged

Clean CLI integration #22

merged 5 commits into from
Nov 27, 2024

Conversation

thomaslepoix
Copy link
Member

No description provided.

@thomaslepoix thomaslepoix added the CLI Command Line Interface label Nov 26, 2024
@thomaslepoix thomaslepoix added this to the 0.0.1 milestone Nov 26, 2024
@thomaslepoix thomaslepoix mentioned this pull request Nov 26, 2024
2 tasks
//******************************************************************************
void OpenEMSH::parse() {
board = ParserFromCsx::run(params.input, dynamic_cast<ParserFromCsx::Params const&>(params));
board->params = dynamic_cast<domain::Params const&>(params);

Check warning

Code scanning / CodeQL

Slicing Warning

This assignment expression slices from type
Params
to
Params
.
SerializerToCsx::run(*board, params.input, params.output, dynamic_cast<SerializerToCsx::Params const&>(params));
break;
case Params::OutputFormat::PLANTUML: {
// SerializerToPlantuml::run(*board, dynamic_cast<SerializerToPlantuml::Params const&>(params));

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.
app.add_flag("-v,--verbose", params.verbose, "Verbose mode.")->capture_default_str();
app.add_flag("-G", params.gui, "GUI mode.");
app.add_option("-i,--input", params.input, "Input CSX file.")->check(CLI::ExistingFile)->required();
// app.add_option("-o,--output", params.output, "Output CSX file. If different from input, will copy and extend it.")->check((!CLI::ExistingFile)|FutureConditional(params.force,"Cannot overwrite a file without --force"));

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.

app.add_flag("--no-x", [&params](size_t) { params.with_axis_x = false; }, "Don't include X axis meshlines in output.")->group("Output options");
app.add_flag("--no-y", [&params](size_t) { params.with_axis_y = false; }, "Don't include Y axis meshlines in output.")->group("Output options");
app.add_flag("--no-z", [&params](size_t) { params.with_axis_z = false; }, "Don't include Z axis meshlines in output.")->group("Output options");
app.add_flag("--meshlines", params.with_meshlines, "Include regular meshlines in output.")->group("Output options")->default_str(to_string(params.with_meshlines));
app.add_flag("--policy-lines", params.with_meshline_policies, "Include meshline policies in output.")->group("Output options")->default_str(to_string(params.with_meshline_policies));
// app.add_flag("--policy-lines", params.with_meshline_policies, "Include meshline policies in output.")->group("Output options")->capture_default_str();

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.
@thomaslepoix thomaslepoix merged commit bfd7848 into main Nov 27, 2024
2 of 3 checks passed
@thomaslepoix thomaslepoix deleted the dev/complete-cli branch November 27, 2024 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI Command Line Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant