Skip to content

General CLI overhaul#44

Merged
facundoolano merged 28 commits into
mainfrom
commands_overhaul
Jun 5, 2021
Merged

General CLI overhaul#44
facundoolano merged 28 commits into
mainfrom
commands_overhaul

Conversation

@facundoolano
Copy link
Copy Markdown
Owner

@facundoolano facundoolano commented Jun 5, 2021 •
edited
Loading

  • Arrange functionality to work around clap subcommands.
  • Improve control over output.
  • Update docs.

facundoolano added 28 commits June 4, 2021 21:22
@facundoolano
rewrite clap opts to use subcommands
14cc8fd
@facundoolano
rewrite using match
adfe564
@facundoolano
handle default destination from clap argument
e34a1c3
@facundoolano
cleanup braces
7ce6f19
@facundoolano
remove cmd option, force stat default
1e12d35
@facundoolano
handle cd -f together with the regular cd
bb01c61
@facundoolano
subcommand shorthands
148f74e
@facundoolano
add short status option
4ecfe81
@facundoolano
added plain status option
862a637
@facundoolano
improve the help
1b96739
@facundoolano
fix ci test
b05063c
@facundoolano
refactor location.go_to and add game.visit
5c587c7
@facundoolano
preserve destination side-effects on cd -f
06b4bb3
@facundoolano
set quiet/plain as (still unused) globals
41d3791
@facundoolano
global quiet/plain logging with once_cell
cdc1710
@facundoolano
use quiet/plain in battle
fd8a82f
@facundoolano
quiet/plain should be global commands
dbd63ea
@facundoolano
more output cleanup
388b0aa
@facundoolano
improve tombstone output
ef73845
@facundoolano
another tombstone improvement
d8dbd0d
@facundoolano
no battles at home
c466aba
@facundoolano
true up readme examples
678a61d
@facundoolano
update changelog
16f8f68
@facundoolano
use cd -f for ci to prevent error exit code
d013b2e
@facundoolano
add --force to cargo install example
7663272
@facundoolano
remove redundant help
4b9e185
@facundoolano
clap help improvements
7be8ad5
@facundoolano
format
21a354c
@facundoolano facundoolano merged commit f962cc7 into main Jun 5, 2021
@facundoolano facundoolano deleted the commands_overhaul branch June 5, 2021 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@facundoolano