Skip to content

Fix fulldiag_evaluate help and license output (fixes bug #128) - #131

Merged
skilledwolf merged 1 commit into
ALPSim:masterfrom
skilledwolf:WIP/fix-fulldiag-evaluate-cli
Aug 26, 2026
Merged

Fix fulldiag_evaluate help and license output (fixes bug #128)#131
skilledwolf merged 1 commit into
ALPSim:masterfrom
skilledwolf:WIP/fix-fulldiag-evaluate-cli

Conversation

@skilledwolf

Copy link
Copy Markdown
Collaborator

Summary

  • add working --help/-h and --license/-l options
  • print the standard ALPS copyright and license information
  • report malformed options cleanly while preserving generic --NAME value parameters
  • add focused CTest coverage for help, license, and no-argument behavior

Testing

  • cmake --build --preset default --target fulldiag_evaluate --parallel 4
  • ctest --test-dir build/default --output-on-failure -R ^fulldiag_evaluate

Closes #128

@skilledwolf skilledwolf changed the title Fix fulldiag_evaluate help and license output Fix fulldiag_evaluate help and license output (fixes bug #128) Aug 20, 2026
@skilledwolf
skilledwolf marked this pull request as ready for review August 20, 2026 21:29
@skilledwolf skilledwolf self-assigned this Aug 20, 2026

@egull egull left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thanks!

@skilledwolf

Copy link
Copy Markdown
Collaborator Author

@marcusr2ML do you have some capacity to take a look at this? it's just a a few lines of code to review -- it fixes how the command line arguments are parsed and prints license information.

@marcusr2ML

Copy link
Copy Markdown
Collaborator

Sorry, missed this.

@skilledwolf
skilledwolf merged commit 4f51b39 into ALPSim:master Aug 26, 2026
39 checks passed
@skilledwolf
skilledwolf deleted the WIP/fix-fulldiag-evaluate-cli branch August 26, 2026 19:58
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.

[Bug]: fulldiag_evaluate does not print info

3 participants