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

Fix CLI description printout #1566

Merged
merged 2 commits into from
Dec 27, 2023
Merged

Fix CLI description printout #1566

merged 2 commits into from
Dec 27, 2023

Conversation

opotowsky
Copy link
Member

@opotowsky opotowsky commented Dec 27, 2023

What is the change?

Splits what prints of the description for the ARMI CLI at the impl tag.

Why is the change being made?

Because the full docstring was being used, the impl tag was being printed when you run armi -h:

image

This PR fixes that to:

image


Checklist

  • The release notes (location doc/release/0.X.rst) are up-to-date with any important changes.
  • The documentation is still up-to-date in the doc folder.
  • If any requirements were affected, mention it in the release notes.
  • The dependencies are still up-to-date in pyproject.toml.

@john-science
Copy link
Member

Approved!

Thanks!

@john-science john-science merged commit dc78053 into main Dec 27, 2023
21 checks passed
@john-science john-science deleted the cli-fix branch December 27, 2023 19:28
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.

2 participants