Skip to content

Add man page; improve CLI ergonomics; simplify config docs - #9

Merged
firefly-cpp merged 7 commits into
firefly-cpp:masterfrom
musicinmybrain:man-etc
Nov 17, 2021
Merged

Add man page; improve CLI ergonomics; simplify config docs#9
firefly-cpp merged 7 commits into
firefly-cpp:masterfrom
musicinmybrain:man-etc

Conversation

@musicinmybrain

Copy link
Copy Markdown
Contributor

As requested in #8, this PR contains a man page.

It also contains a few typo fixes.

I suggested a couple of compatible improvements to the command-line option parsing that I think would make using the command simpler and less surprising.

In the final commit, I heavily edited the configuration file documentation for simplicity and clarity. You didn’t ask me to do that, and I am happy to revert or further modify this commit if you’re not happy with the result. However, I think there is a good chance that you will find it to be an improvement.

I’m happy to accept any feedback and make further revisions if needed.

One instance of “licence“ is changed to match the predominant spelling
variant “license.”
While “uARMSolver arm.set” would work, it would do so only because
“arm.set” is the default setup file name/path, and extra positional
arguments are ignored.
This makes the command-line interface a little more conventional and
ergonomic.
This is much less confusing than silently ignoring them.
@firefly-cpp

Copy link
Copy Markdown
Owner

Ben, this is brilliant! Great! Many thanks.

@firefly-cpp
firefly-cpp merged commit 9635270 into firefly-cpp:master Nov 17, 2021
@firefly-cpp firefly-cpp mentioned this pull request Dec 2, 2021
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